]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Force L3 agent to resync router it could not configure
authorAssaf Muller <amuller@redhat.com>
Sat, 19 Dec 2015 19:13:43 +0000 (14:13 -0500)
committerAssaf Muller <amuller@redhat.com>
Sat, 19 Dec 2015 19:51:47 +0000 (14:51 -0500)
commit822ad5f06bcef8f95f36032d4fd4709975cecc31
tree7760abf187a5be0de0531b735467366ab24e8d57
parent495f8c606e923a758645692045cddf070b6a7a7f
Force L3 agent to resync router it could not configure

If the L3 agent fails to configure a router, commit:
4957b5b43521a61873a041fe3e8989ed399903d9 changed it so
that instead of performing an expensive full sync, only that
router is reconfigured. However, it tries to reconfigure the
cached router. This is a change of behavior from the fullsync
days. The retry is more likely to succeed if the
router is retrieved from the server, instead of using
the locally cached version, in case the user or operator
fixed bad input, or if the router was retrieved in a bad
state due to a server-side race condition.

Note that this is only relevant to full syncs, as those retrieve
routers from the server and queue updates with the router object.
Incremental updates queue up updates without router objects,
so if one of those fails it would always be resynced on a
second attempt.

Related-Bug: #1494682
Change-Id: Id0565e11b3023a639589f2734488029f194e2f9d
neutron/agent/l3/agent.py
neutron/tests/unit/agent/l3/test_agent.py