DVR l3 agent should check for snat namespace existence before adding or deleting anything from the namespace

Bug #1564776 reported by Swaminathan Vasudevan on 2016-04-01
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
neutron
Undecided
Swaminathan Vasudevan

Bug Description

Check for snat_namespace existence in the node before any operation
in the namespace.

Today we check the self.snatnamespace which may or may not reflect
the exact state of the system.

If the snat_namespace is accidentally deleted and if we try to
remove the gateway from the router, the agent throws in a bunch of
error messages and the agent goes in loop constantly spewing error
messages.

Here is the link to the error message.

http://paste.openstack.org/show/492700/

This can be easily reproduced.

1. Create a network
2. Create a subnet
3. Create a router ( dvr)
4. Attach the subnet to the router.
5. Configure default gateway to the router.
6. Now verify the namespaces in the 'dvr_snat' node.
7. You should see
    a. snat_namespace
    b. router_namespace
    c. dhcp namespace.
8. Now delete the snat_namespace.
9. Try to remove the gateway from the router.
10. Watch the L3 agent logs

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

Changed in neutron:
assignee: nobody → Swaminathan Vasudevan (swaminathan-vasudevan)
status: New → In Progress
Brian Haley (brian-haley) wrote :

I think this was all fixed by https://review.openstack.org/#/c/326729/ so can probably abandon patch and close this bug.

Yes you are right, we can close this bug, since multiple fixes to address in this area have already merged.

Change abandoned by Swaminathan Vasudevan (<email address hidden>) on branch: master
Review: https://review.openstack.org/300358
Reason: Multiple patches that addressed the issues with this bug have already merged. So will adbandon this patch and close this bug.

This bug can be closed, since alternate patches have merged upstream that addressed the same problem.

Changed in neutron:
status: In Progress → Invalid
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers