]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
L3 Agent should generate ns_name in a single place
authorJohn Schwarz <jschwarz@redhat.com>
Tue, 23 Sep 2014 11:41:54 +0000 (14:41 +0300)
committerJohn Schwarz <jschwarz@redhat.com>
Sun, 5 Oct 2014 10:29:21 +0000 (13:29 +0300)
commit467bd9476defbb028f0fbfa26f01f76c32b701cd
tree6f248de93567e07a6beb48c8fe20cb5f36a4dc50
parent205162f58050fcb94db53cd51b674d2093dfe700
L3 Agent should generate ns_name in a single place

Currently the l3 agent has 2 places where it allows generating ns_name
of specific router_ids (ie. qrouter-<router_id>): in the RouterInfo's
constructor, and in _cleanup_namespaces. This patch proposes a
unification of this creation code with a property which lives in
RouterInfo's namespace. A simpler fix was also made for snat_ns_name.

This patch also offers a single way to initialize a new RouterInfo.

Related-bug: #1374946
Related-bug: #1374947
Change-Id: Ia028236b73a22ff534acee00b46c86b134dc987e
neutron/agent/l3_agent.py