]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Static routes not added to qrouter namespace for DVR
authorSwaminathan Vasudevan <swaminathan.vasudevan@hp.com>
Fri, 25 Sep 2015 16:54:44 +0000 (09:54 -0700)
committerArmando Migliaccio <armamig@gmail.com>
Wed, 13 Jan 2016 18:57:35 +0000 (18:57 +0000)
commit158f9eabe20824b2c91eaac795dad8b8a773611d
treed5f6fbb6eb783bc3e42e452d10235a6e4b82f863
parentb600129553f92003c0f79ff8005d83067323fb9e
Static routes not added to qrouter namespace for DVR

Today static routes are added to the SNAT namespace
for DVR routers. But they are not added to the qrouter
namespace.

Also while configuring the static routes to SNAT
namespace, the router is not checked for the existence
of the gateway.

When routes are added to a router without a gateway the
routes are only configured in the router namespace, but
when a gateway is set later, those routes have to be
populated in the snat_namespace as well.

This patch addresses the above mentioned issues.

Closes-Bug: #1499785
Closes-Bug: #1499787

Change-Id: I37e0d0d723fcc727faa09028045b776957c75a82
neutron/agent/l3/dvr_edge_router.py
neutron/agent/l3/ha_router.py
neutron/agent/l3/router_info.py
neutron/tests/functional/agent/l3/framework.py
neutron/tests/functional/agent/l3/test_dvr_router.py
neutron/tests/unit/agent/l3/test_agent.py
neutron/tests/unit/agent/l3/test_router_info.py