OpenContrail

SYMC: Incorrect default route created on some networks

Bug #1483456 reported by Rudrajit Tapadar
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.0
Fix Committed
High
Sachin Bansal
Juniper Openstack r2.02
R2.20
Fix Committed
High
Sachin Bansal
Juniper Openstack r2.21
Trunk
Fix Committed
High
Sachin Bansal
Juniper Openstack r3.0-fcs
OpenContrail
Fix Committed
High
Sachin Bansal

Bug Description

We observed that several networks are getting created with incorrect default routes attached to snat instances. On investigation, we found that the snat instances had references to the the route target's that belonged to the affected networks. This is possibly due to reuse of route-targets and a cleanup not happening in the database.

Tags: config snat
Rudrajit Tapadar (rtapadar)
summary: - Incorrect default route getting created on networks
+ Incorrect default route created on some networks
Conrad Dias (conrad-dias)
summary: - Incorrect default route created on some networks
+ SYMC: Incorrect default route created on some networks
Nagabhushana R (bhushana)
Changed in juniperopenstack:
importance: Undecided → High
assignee: nobody → Rudra Rugge (rudrarugge)
Changed in opencontrail:
importance: Undecided → High
assignee: nobody → Rudra Rugge (rudrarugge)
tags: added: config
tags: added: snat
Sachin Bansal (sbansal)
Changed in opencontrail:
assignee: Rudra Rugge (rudrarugge) → Sachin Bansal (sbansal)
Nischal Sheth (nsheth)
Changed in juniperopenstack:
assignee: Rudra Rugge (rudrarugge) → Sachin Bansal (sbansal)
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/12986
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20

Review in progress for https://review.opencontrail.org/12988
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.0

Review in progress for https://review.opencontrail.org/12989
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/12986
Committed: http://github.org/Juniper/contrail-controller/commit/e9fda7d318ab0b9076da3d3c94f24bbe83063974
Submitter: Zuul
Branch: master

commit e9fda7d318ab0b9076da3d3c94f24bbe83063974
Author: Sachin Bansal <email address hidden>
Date: Tue Aug 11 15:59:21 2015 -0700

Remove static routes if a VN is deleted with exising ref to route table

If we received networn delete before its link to route table is deleted, we were
not cleaning up corresponding static routes. With this change, we take care of
it. Also add a unit test case for it.

Change-Id: Iafbb808a0a12e0523bc79a9aa46adf15794f9ceb
Closes-Bug: 1483456
(cherry picked from commit 6db79ac72c901dbec8e71196dd9afd20bb6eb4ac)

Nischal Sheth (nsheth)
Changed in opencontrail:
status: New → Fix Committed
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/12988
Committed: http://github.org/Juniper/contrail-controller/commit/242e639c4271b1990296a1d3797d6f414b85a5c3
Submitter: Zuul
Branch: R2.20

commit 242e639c4271b1990296a1d3797d6f414b85a5c3
Author: Sachin Bansal <email address hidden>
Date: Tue Aug 11 15:59:21 2015 -0700

Remove static routes if a VN is deleted with exising ref to route table

If we received networn delete before its link to route table is deleted, we were
not cleaning up corresponding static routes. With this change, we take care of
it. Also add a unit test case for it.

Change-Id: Iafbb808a0a12e0523bc79a9aa46adf15794f9ceb
Closes-Bug: 1483456
(cherry picked from commit 6db79ac72c901dbec8e71196dd9afd20bb6eb4ac)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/12989
Committed: http://github.org/Juniper/contrail-controller/commit/cedbf382274b73c63fb3cde7b907a66275583b5f
Submitter: Zuul
Branch: R2.0

commit cedbf382274b73c63fb3cde7b907a66275583b5f
Author: Sachin Bansal <email address hidden>
Date: Tue Aug 11 15:59:21 2015 -0700

Remove static routes if a VN is deleted with exising ref to route table

If we received networn delete before its link to route table is deleted, we were
not cleaning up corresponding static routes. With this change, we take care of
it. Also add a unit test case for it.

Change-Id: Iafbb808a0a12e0523bc79a9aa46adf15794f9ceb
Closes-Bug: 1483456
(cherry picked from commit 6db79ac72c901dbec8e71196dd9afd20bb6eb4ac)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.22-dev

Review in progress for https://review.opencontrail.org/13927
Submitter: Vinay Vithal Mahuli (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20.x

Review in progress for https://review.opencontrail.org/14203
Submitter: Hampapur Ajay (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/14203
Committed: http://github.org/Juniper/contrail-controller/commit/17f92d47b91fc6cdd31aef04f8e6de66f24c323e
Submitter: Zuul
Branch: R2.20.x

commit 17f92d47b91fc6cdd31aef04f8e6de66f24c323e
Author: Sachin Bansal <email address hidden>
Date: Tue Aug 11 15:59:21 2015 -0700

Remove static routes if a VN is deleted with exising ref to route table

If we received networn delete before its link to route table is deleted, we were
not cleaning up corresponding static routes. With this change, we take care of
it. Also add a unit test case for it.

Change-Id: Iafbb808a0a12e0523bc79a9aa46adf15794f9ceb
Closes-Bug: 1483456
(cherry picked from commit 6db79ac72c901dbec8e71196dd9afd20bb6eb4ac)
(cherry picked from commit 242e639c4271b1990296a1d3797d6f414b85a5c3)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.