Static routes are not added to the qrouter namespace for DVR routers

[Swaminathan Vasudevan]

Static routes are not added to the qrouter namespace when routers are added.

Initially it used to be configuring the routes in the qrouter namespace but not in the SNAT namespace.
A recent patch caused this regression in moving the routes from qrouter namespace to SNAT namespace.

2bb48eb58ad28a629dd12c434b83680aa3f240a4

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/228026

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/228026
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=158f9eabe20824b2c91eaac795dad8b8a773611d
Submitter: Jenkins
Branch: master

commit 158f9eabe20824b2c91eaac795dad8b8a773611d
Author: Swaminathan Vasudevan <email address hidden>
Date: Fri Sep 25 09:54:44 2015 -0700

    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

[Thierry Carrez (ttx)]

This issue was fixed in the openstack/neutron 8.0.0.0b2 development milestone.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/275758

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/275758
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=4bb364044bb99f559c565f51db756f5ee9ac73b7
Submitter: Jenkins
Branch: stable/liberty

commit 4bb364044bb99f559c565f51db756f5ee9ac73b7
Author: Swaminathan Vasudevan <email address hidden>
Date: Fri Sep 25 09:54:44 2015 -0700

    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

    Conflicts:
     neutron/agent/l3/dvr_edge_router.py
            neutron/tests/functional/agent/test_l3_agent.py
     neutron/tests/functional/agent/l3/framework.py
     neutron/tests/functional/agent/l3/test_dvr_router.py

    Change-Id: I37e0d0d723fcc727faa09028045b776957c75a82
    (cherry picked from commit 158f9eabe20824b2c91eaac795dad8b8a773611d)

[Doug Hellmann (doug-hellmann)]

This issue was fixed in the openstack/neutron 7.0.4 release.