Skip extra logging when devices is empty

Change-Id: Id9ae8aef0555109bd88600965adb2744ab924430
Closes-Bug: #1278271
This commit is contained in:
Yuanchao Sun 2014-02-10 00:20:31 +08:00
parent d8c105f00c
commit 26d8231b0b

View File

@ -150,13 +150,14 @@ class SecurityGroupAgentRpcMixin(object):
for device in self.firewall.ports.values(): for device in self.firewall.ports.values():
if sec_grp_set & set(device.get(attribute, [])): if sec_grp_set & set(device.get(attribute, [])):
devices.append(device['device']) devices.append(device['device'])
if self.defer_refresh_firewall: if devices:
LOG.debug(_("Adding %s devices to the list of devices " if self.defer_refresh_firewall:
"for which firewall needs to be refreshed"), LOG.debug(_("Adding %s devices to the list of devices "
devices) "for which firewall needs to be refreshed"),
self.devices_to_refilter |= set(devices) devices)
elif devices: self.devices_to_refilter |= set(devices)
self.refresh_firewall(devices) else:
self.refresh_firewall(devices)
def security_groups_provider_updated(self): def security_groups_provider_updated(self):
LOG.info(_("Provider rule updated")) LOG.info(_("Provider rule updated"))