Merge "Skip extra logging when devices is empty"

This commit is contained in:
Jenkins 2014-02-12 04:01:57 +00:00 committed by Gerrit Code Review
commit f21dad0579

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"))