Send only one agent notification on port update

This patch removes the other redundant notification sent in the same
routine to notify whether a security group changed.

Closes-Bug: #1254924
Partial Blueprint neutron-tempest-parallel

Change-Id: I5bad015decdc197c3fa9b58c680564380d513bc4
This commit is contained in:
Salvatore Orlando 2013-11-25 16:25:43 -08:00
parent 15d5f1618c
commit 09c3051fb2
2 changed files with 1 additions and 5 deletions

View File

@ -98,8 +98,6 @@ class SecurityGroupServerRpcMixin(sg_db.SecurityGroupDbMixin):
not utils.compare_elements(
original_port.get(ext_sg.SECURITYGROUPS),
updated_port.get(ext_sg.SECURITYGROUPS))):
self.notify_security_groups_member_updated(
context, updated_port)
need_notify = True
return need_notify

View File

@ -1442,9 +1442,7 @@ class SGNotificationTestMixin():
self._delete('ports', port['port']['id'])
self.notifier.assert_has_calls(
[call.security_groups_member_updated(
mock.ANY, [mock.ANY]),
call.security_groups_member_updated(
mock.ANY, [security_group_id])])
mock.ANY, [mock.ANY])])
class TestSecurityGroupAgentWithOVSIptables(