x/vmware-nsx
Code Issues Proposed changes

Rebind security groups only when they're updated

Browse Source 
Update the security port bindings for a port only when they actually
differ from the stored value.

This will also avoid sending port_update notifications to the agent
even if nothing actually changes in the port configuration.

Closes-Bug: #1255183
Partial blueprint: neutron-tempest-parallel

Change-Id: I00c29dc97c46478433fdf08069a884bb78e5cd0a
This commit is contained in:
Salvatore Orlando 2013-11-26 09:53:49 -08:00
parent 1811ea2759
commit 598ca24470
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
neutron/db/securitygroups_rpc_base.py
View File

@ -70,15 +70,19 @@ class SecurityGroupServerRpcMixin(sg_db.SecurityGroupDbMixin):
It is because another changes for the port may require notification. It is because another changes for the port may require notification.
""" """
need_notify = False need_notify = False
if ext_sg.SECURITYGROUPS in port['port']: port_updates = port['port']
if (ext_sg.SECURITYGROUPS in port_updates and
not utils.compare_elements(
original_port.get(ext_sg.SECURITYGROUPS),
port_updates[ext_sg.SECURITYGROUPS])):
# delete the port binding and read it with the new rules # delete the port binding and read it with the new rules
port['port'][ext_sg.SECURITYGROUPS] = ( port_updates[ext_sg.SECURITYGROUPS] = (
self._get_security_groups_on_port(context, port)) self._get_security_groups_on_port(context, port))
self._delete_port_security_group_bindings(context, id) self._delete_port_security_group_bindings(context, id)
self._process_port_create_security_group( self._process_port_create_security_group(
context, context,
updated_port, updated_port,
port['port'][ext_sg.SECURITYGROUPS]) port_updates[ext_sg.SECURITYGROUPS])
need_notify = True need_notify = True
else: else:
updated_port[ext_sg.SECURITYGROUPS] = ( updated_port[ext_sg.SECURITYGROUPS] = (