[SRU] When router has no ports _process_updated_router fails because the namespace does not exist
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ubuntu Cloud Archive |
Fix Released
|
High
|
Unassigned | ||
Liberty |
Fix Released
|
High
|
Unassigned | ||
Mitaka |
Fix Released
|
High
|
Unassigned | ||
Newton |
Fix Released
|
High
|
Unassigned | ||
neutron |
Fix Released
|
Undecided
|
Unassigned | ||
neutron (Ubuntu) |
Fix Released
|
High
|
Unassigned | ||
Xenial |
Fix Released
|
High
|
Unassigned | ||
Yakkety |
Fix Released
|
High
|
Unassigned | ||
Zesty |
Fix Released
|
High
|
Unassigned |
Bug Description
[Description]
[Testcase]
Happens in Kilo. Cannot test on other releases.
Steps to reproduce:
1) create a router and set at least a port, also the gateway is fine
2) check that the namespace exists with
ip netns show | grep qrouter-<uuid>
3) check the ports are there
ip netns exec qrouter-<uuid> ip addr show
4) delete all ports from the router
5) check that only loopback interface is present
ip netns exec qrouter-<uuid> ip addr show
6) run the cronjob task that is installed in the file
/etc/
so basically run this command:
/usr/
7) the namespace should be gone:
ip netns show | grep qrouter-<uuid>
8) delete the neutron router.
9) check log file /var/log/
When the router has no ports the namespace is deleted from the network node by the cronjob. However this brakes the router updates and the file vpn-agent.log is flooded with this traces:
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.771 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 ERROR neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
2016-04-21 16:22:17.774 23382 TRACE neutron.
[Regression Potential]
The patches that we're picking up in our packages have already landed upstream in master (zesty), stable/newton (yakkety), and stable/mitaka (xenial).
Related branches
- Corey Bryant: Pending requested
-
Diff: 112 lines (+90/-0)3 files modifieddebian/changelog (+7/-0)
debian/patches/ns-exists-before-get-devices.patch (+82/-0)
debian/patches/series (+1/-0)
Changed in neutron: | |
status: | New → In Progress |
description: | updated |
Changed in neutron (Ubuntu Zesty): | |
importance: | Undecided → High |
Changed in neutron (Ubuntu Yakkety): | |
importance: | Undecided → High |
Changed in neutron (Ubuntu Xenial): | |
importance: | Undecided → High |
status: | New → Triaged |
Changed in neutron (Ubuntu Zesty): | |
status: | New → Triaged |
Changed in neutron (Ubuntu Yakkety): | |
status: | New → Triaged |
Changed in cloud-archive: | |
status: | Triaged → Fix Committed |
Changed in neutron: | |
status: | In Progress → Fix Released |
Related fix proposed to branch: master /review. openstack. org/309050
Review: https:/