]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Code Review / openstack-build / neutron-build.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 7df578c) | patch
Add OVS status and fix OVS crash
authorRobinWang <wangbibo@huawei.com>
Tue, 9 Dec 2014 03:52:14 +0000 (11:52 +0800)
committerRobinWang <wangbibo@huawei.com>
Wed, 17 Dec 2014 15:23:03 +0000 (23:23 +0800)
commit91b7fc7f162751936f7cb15d4add932a4aebd55b
tree03855777c903f105edd62b66d2fb36b6c133b79dtree | snapshot
parent7df578c4512adbd2443114b11b83589a22bba19fcommit | diff
Add OVS status and fix OVS crash

OVS crash/restart is unpredictable, so neutron-ovs-agent should be
robust enough under that situation. But currently ovs-agent doesn't
figure out this error status(only check ovs restart/normal status)
and still continue to apply subsequent operations(set br/add patch
port/...) till causing exceptions/crash. Add flag to fully represent
ovs status. Base on that, we can add proper fail-over code in method
rpc_loop, to treat ovs dead/restart gracefully to prevent agent
crashes while it is running.
Closes-bug: #1296202

Change-Id: Id058b7ddef2ed337627dc692d0418786ad14f4b4
neutron/plugins/openvswitch/agent/ovs_neutron_agent.py diff | blob | history
neutron/plugins/openvswitch/common/constants.py diff | blob | history
neutron/tests/unit/openvswitch/test_ovs_neutron_agent.py diff | blob | history