Neutron sidecar wrappers make assumptions about netns names

[Brent Eagles]

The wrappers for launching sidecar containers for the L3 agent are using the incorrect namespace when launching keepalived etc.

[OpenStack Infra (hudson-openstack)]

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

[Brent Eagles (beagles)]

While the patch is a nice improvement to how wrappers figure out what namespace to run the sidecar in, this may not actually be an issue as the problem showed up when I accidentally create a router that was both HA and distributed - which IIRC is not currently supported.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/570819
Committed: https://git.openstack.org/cgit/openstack/puppet-tripleo/commit/?id=684521fc84bdf589de67972ac6f263b5e3bc24a4
Submitter: Zuul
Branch: master

commit 684521fc84bdf589de67972ac6f263b5e3bc24a4
Author: Brent Eagles <email address hidden>
Date: Mon May 28 13:43:03 2018 -0230

    Neutron sidecar wrappers use netns they were invoked with

    Neutron uses namespaces with different prefixes depending on
    configuration and the nature of the resource. This patch changes the
    wrappers to use the "ip netns identify" command to determine the target
    namespace for the sidecar instead of trying to guess from the command
    line options.

    Change-Id: If58bb9dabebf201b592fb450a663ae2f24374e00
    Closes-Bug: #1773823

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/queens
Review: https://review.openstack.org/571484

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/puppet-tripleo 9.1.0 release.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/571484
Committed: https://git.openstack.org/cgit/openstack/puppet-tripleo/commit/?id=a8de6dc7929f79289f3ee0d74971c1b9db84103e
Submitter: Zuul
Branch: stable/queens

commit a8de6dc7929f79289f3ee0d74971c1b9db84103e
Author: Brent Eagles <email address hidden>
Date: Mon May 28 13:43:03 2018 -0230

    Neutron sidecar wrappers use netns they were invoked with

    Neutron uses namespaces with different prefixes depending on
    configuration and the nature of the resource. This patch changes the
    wrappers to use the "ip netns identify" command to determine the target
    namespace for the sidecar instead of trying to guess from the command
    line options.

    Change-Id: If58bb9dabebf201b592fb450a663ae2f24374e00
    Closes-Bug: #1773823
    (cherry picked from commit 684521fc84bdf589de67972ac6f263b5e3bc24a4)

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/puppet-tripleo 8.3.4 release.