]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Improve exception handling in _process_router_update()
authorOleg Bondarev <obondarev@mirantis.com>
Fri, 20 Feb 2015 14:11:44 +0000 (17:11 +0300)
committerOleg Bondarev <obondarev@mirantis.com>
Fri, 20 Feb 2015 14:32:32 +0000 (17:32 +0300)
commit4f55c8f689861631a8060e9918066792f56dfcf5
treec590fbf5aa26c7989871f413c679678d085e45b3
parent4b11655b95ec35ae8ca426c0a557a0f2883010d1
Improve exception handling in _process_router_update()

in _process_router_update() where _process_router_if_compatible()
method is called only RouterNotCompatibleWithAgent exception is handled.
In case any other (intermittent) exception happens inside
_process_router_if_compatible() (e.g. MessagingTimeout on fetching
external net from server) it results in a situation where agent completely
forgets about the router and continues working as usual while server
shows router hosted by agent.

This patch adds handler for broader exception there and set
fullsync = True like it's done earlier in _process_router_update()
when getting routers from server.

Change-Id: I5bfc540a926349848702100744f97c597b852604
Closes-Bug: #1423571
neutron/agent/l3/agent.py
neutron/tests/unit/test_l3_agent.py