]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
manual add/remove router for dvr_snat agent
authorMichael Smith <michael.smith6@hp.com>
Wed, 17 Sep 2014 00:00:05 +0000 (17:00 -0700)
committerOleg Bondarev <obondarev@mirantis.com>
Tue, 18 Aug 2015 14:22:27 +0000 (17:22 +0300)
commit7cfcbac066842e2d1dd3578e9eb5801bb4a1af34
tree57ebc36f6206a9d7d4f99004cac2dc5cad05f57b
parenta264b329df82f3c391f75bd9ad73a1327616a43d
manual add/remove router for dvr_snat agent

This patch is to address the failure of manual move of
dvr_snat routers from one service node to another.

The entry in the csnat_l3_agent_bindings table is now removed
during the router to agent unbind operation.
Appropriate notification is now sent to the agent to remove
snat/qrouter namespace.

There were other places in the code
that needed to examine the snat binding table to
check if updates were required -
validate_agent_router_combination() and
check_agent_router_scheduling_needed().

Additionally, schedule_routers() was made optional
within the rpc _notification path since it can
override the manual move being attempted.

Change-Id: Iac9598eb79f455c4ef3d3243a96bed524e3d2f7c
Closes-Bug: #1369721
Co-Authored-By: Ila Palanisamy <ilavajuthy.palanisamy@hp.com>
Co-Authored-By: Oleg Bondarev <obondarev@mirantis.com>
neutron/api/rpc/agentnotifiers/l3_rpc_agent_api.py
neutron/api/rpc/handlers/l3_rpc.py
neutron/db/l3_agentschedulers_db.py
neutron/db/l3_dvrscheduler_db.py
neutron/tests/unit/plugins/ml2/drivers/openvswitch/agent/test_agent_scheduler.py