]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Improve handling of security group updates
authorSalvatore Orlando <salv.orlando@gmail.com>
Thu, 19 Dec 2013 08:26:38 +0000 (00:26 -0800)
committerThomas Goirand <thomas@goirand.fr>
Thu, 13 Mar 2014 07:20:18 +0000 (15:20 +0800)
commit3276658a69a58496e57da571fce07982a65ab528
treee78d2425db3b4553a1ee996222976cb355e5e043
parentc38ef8aa18eb5db46ab442208ceb181e17879f31
Improve handling of security group updates

Currently updates to security group rules or membership
are handled by immediately triggering a call to refresh_firewall.
This call is quite expensive, and it is often executed with a
very high frequency.

With this patch, the notification handler simply adds devices for
which the firewall should be refreshed to a set, which will then
be processed in another routine. The latter is supposed to
be called in the main agent loop.

This patch for 'provider updates' simply sets a flag for refreshing
the firewall for all devices.

In order to avoid breaking other agents leveraging the security
group RPC mixin, the reactive behaviour is still available, and is
still the default way of handling security group updates.

Partial-Bug: #1253993
Partially implements blueprint: neutron-tempest-parallel

Change-Id: I1574544734865506ff5383404516cc9349c16ec4
neutron/agent/securitygroups_rpc.py
neutron/plugins/openvswitch/agent/ovs_neutron_agent.py
neutron/tests/unit/openvswitch/test_ovs_neutron_agent.py
neutron/tests/unit/test_security_groups_rpc.py