]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Prevent possible server list damage in BigSwitch plugin
authorKevin Benton <kevin.benton@bigswitch.com>
Wed, 10 Jul 2013 23:28:12 +0000 (16:28 -0700)
committerKevin Benton <kevin.benton@bigswitch.com>
Sun, 14 Jul 2013 03:07:48 +0000 (20:07 -0700)
commit3732f022778a7c4a47c58029ae77e32a0a4a6ee9
treea71d56c20c4dec4e6abf2a0c1ae949763d934899
parentcac2612f2f9c7787e827b0f5e1e163d995a359e0
Prevent possible server list damage in BigSwitch plugin

The old failover logic for the BigSwitch plugin modified a
list of controllers as it tested them. If the code for one
thread unexpectedly raised an exception or died, the global
server list could lose a server permanently.
This patch addresses that by flagging servers as failed instead
so the global server list is never modified.

Fixes: bug #1200022
Change-Id: Id2dcb820ef9f62fd03e3215bff3345e56c78afe2
neutron/plugins/bigswitch/plugin.py
neutron/tests/unit/bigswitch/etc/restproxy.ini.test
neutron/tests/unit/bigswitch/test_restproxy_plugin.py