Addition and deletion of Connected VNs for SNAT is broken

Bug #1445692 reported by Sreelakshmi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.0
Fix Committed
High
Balamurugan Gopal
R2.1
Fix Committed
High
Balamurugan Gopal
R2.20
Fix Committed
High
Balamurugan Gopal
Trunk
Fix Committed
High
Balamurugan Gopal

Bug Description

Multiple issues noticed when you add/delete VNs to Connected Virtual networks in our SNAT feature.

1. 0/0 route is not withdrawn when we delete a VN from the connected network.
2. addition of new VN to the connected list is not reflected when you incrementally add one VN after another to an existing Router/SNAT GW.

Tags: blocker config ui
Revision history for this message
Vedamurthy Joshi (vedujoshi) wrote :

Issue 1 happens only when router-configuration via Contrail UI. When done using neutron api/cli, things are fine

Changed in juniperopenstack:
assignee: nobody → Suresh Balineni (sbalineni)
importance: Undecided → High
information type: Proprietary → Public
tags: added: blocker config
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : R2.1

Review in progress for https://review.opencontrail.org/9470
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/9492
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/9470
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/9492
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/9470
Submitter: Balamurugan Gopal (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/9470
Committed: http://github.org/Juniper/contrail-web-controller/commit/ecc3bb2cc57ab4e90b1adea9ee8131145d49c4a6
Submitter: Zuul
Branch: R2.1

commit ecc3bb2cc57ab4e90b1adea9ee8131145d49c4a6
Author: balamurugang <email address hidden>
Date: Fri Apr 24 05:08:26 2015 +0530

In case external gateway for l3 router remains same, but conected VN's are added
or deleted then we need to manually update the route table ref to
"rt_uuid_of_logical_router" in each VN.

In case when external gateway is updated or deleted, then neutron external gateway update
takes care of adding and deleting route table refs for the VN's.

Added error check in the logical router api part..

getting the virtual network using the new api
/api/tenants/config/all-virtual-networks-fields?uuid=
for scaling.

Change-Id: I856aa795b4f5389b6682a925098e724dcdc98702
Closes-Bug: #1445692

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

Review in progress for https://review.opencontrail.org/9532
Submitter: Balamurugan Gopal (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/9532
Committed: http://github.org/Juniper/contrail-web-controller/commit/901f95c406d70ce9d15071384f9b62e15005904c
Submitter: Zuul
Branch: R2.0

commit 901f95c406d70ce9d15071384f9b62e15005904c
Author: balamurugang <email address hidden>
Date: Mon Apr 27 10:36:49 2015 +0530

n case external gateway for l3 router remains same, but conected VN's are added
or deleted then we need to manually update the route table ref to
"rt_uuid_of_logical_router" in each VN.

In case when external gateway is updated or deleted, then neutron external
gateway update
takes care of adding and deleting route table refs for the VN's.

Added error check in the logical router api part..

Change-Id: Ia944999c17935863e3023804d363d53a82db80dc
Closes-Bug: #1445692

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

Review in progress for https://review.opencontrail.org/9677
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/9678
Submitter: Balamurugan Gopal (<email address hidden>)

Nischal Sheth (nsheth)
tags: added: ui
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : R2.1

Review in progress for https://review.opencontrail.org/9926
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/9986
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/9987
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/9988
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/9987
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/10050
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/9677
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/9678
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/9926
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/9986
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/9926
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/9986
Submitter: Balamurugan Gopal (<email address hidden>)

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

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

commit a5d22a6e89a329450ecda3a6f3433b5d42654e6b
Author: balamurugang <email address hidden>
Date: Tue May 5 18:13:21 2015 +0530

Updating logical route ref in network's Route table.

In port page
In case of create and edit if the logical router is set for a port.
The logical router reference has to be added manualley in route table of the
network in the format of rt_uuidOfLogicalRouter.
Same way if the router is detached from the port the reference has to be removed
from the corresponding port's network.
Also if the logical router reference is changed the new reference has to be
updated in the network.
All the case are handled.

The api has got exported from logical router api page and reused in ports page.

Change-Id: I072f444424a4a3f620a83afef19812afb62e6082
Related-Bug: #1445692

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

Reviewed: https://review.opencontrail.org/9926
Committed: http://github.org/Juniper/contrail-web-controller/commit/fcee4b8db0aae42c57c7f0790197b5770ccbc682
Submitter: Zuul
Branch: R2.1

commit fcee4b8db0aae42c57c7f0790197b5770ccbc682
Author: balamurugang <email address hidden>
Date: Tue May 5 18:13:21 2015 +0530

Updating logical route ref in network's Route table.

In port page
In case of create and edit if the logical router is set for a port.
The logical router reference has to be added manualley in route table of the
network in the format of rt_uuidOfLogicalRouter.
Same way if the router is detached from the port the reference has to be removed
from the corresponding port's network.
Also if the logical router reference is changed the new reference has to be
updated in the network.
All the case are handled.

The api has got exported from logical router api page and reused in ports page.

Change-Id: I072f444424a4a3f620a83afef19812afb62e6082
Related-Bug: #1445692

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

Reviewed: https://review.opencontrail.org/9678
Committed: http://github.org/Juniper/contrail-web-controller/commit/769bd69cce7a97ac50616daaf7287f4dfde127da
Submitter: Zuul
Branch: R2.0

commit 769bd69cce7a97ac50616daaf7287f4dfde127da
Author: balamurugang <email address hidden>
Date: Wed Apr 29 18:53:29 2015 +0530

Bug:
In Logical router page of config
At the time of edit When the external gateway is changed to another external
gateway. The nova command fail to set the refference and getting crashed.
Fix:
To resolve the issue the nova command is executed with empty first and then
getting set to the actual value so that nova is not getting crashed.

Change-Id: If01832dab03f9ca7b1ed52492fe458fc414cb061
Related-Bug: #1445692

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

Reviewed: https://review.opencontrail.org/9677
Committed: http://github.org/Juniper/contrail-web-controller/commit/fcde4a30ed26c250c3d177583b5f7af0b97ed6dd
Submitter: Zuul
Branch: R2.1

commit fcde4a30ed26c250c3d177583b5f7af0b97ed6dd
Author: balamurugang <email address hidden>
Date: Wed Apr 29 18:53:29 2015 +0530

Bug:
In Logical router page of config
At the time of edit When the external gateway is changed to another external
gateway. The nova command fail to set the refference and getting crashed.
Fix:
To resolve the issue the nova command is executed with empty first and then
getting set to the actual value so that nova is not getting crashed.

Change-Id: If01832dab03f9ca7b1ed52492fe458fc414cb061
Related-Bug: #1445692

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

Review in progress for https://review.opencontrail.org/10473
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/10520
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/10589
Submitter: Balamurugan Gopal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/10643
Submitter: Balamurugan Gopal (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/10589
Committed: http://github.org/Juniper/contrail-web-controller/commit/cebf30fbcf8968cf5a38a18a26468ccb5e6af60c
Submitter: Zuul
Branch: R2.20

commit cebf30fbcf8968cf5a38a18a26468ccb5e6af60c
Author: balamurugang <email address hidden>
Date: Fri Apr 24 05:08:26 2015 +0530

In case external gateway for l3 router remains same, but conected VN's are added
or deleted then we need to manually update the route table ref to
"rt_uuid_of_logical_router" in each VN.

In case when external gateway is updated or deleted, then neutron external gateway update
takes care of adding and deleting route table refs for the VN's.

Added error check in the logical router api part..

getting the virtual network using the new api
/api/tenants/config/all-virtual-networks-fields?uuid=
for scaling.

Closes-Bug: #1445692
(cherry picked from commit ecc3bb2cc57ab4e90b1adea9ee8131145d49c4a6)

Updating logical route ref in network's Route table.

In port page
In case of create and edit if the logical router is set for a port.
The logical router reference has to be added manualley in route table of the
network in the format of rt_uuidOfLogicalRouter.
Same way if the router is detached from the port the reference has to be removed
from the corresponding port's network.
Also if the logical router reference is changed the new reference has to be
updated in the network.
All the case are handled.

The api has got exported from logical router api page and reused in ports page.

Related-Bug: #1445692
(cherry picked from commit fcee4b8db0aae42c57c7f0790197b5770ccbc682)

Bug:
In Logical router page of config
At the time of edit When the external gateway is changed to another external
gateway. The nova command fail to set the refference and getting crashed.
Fix:
To resolve the issue the nova command is executed with empty first and then
getting set to the actual value so that nova is not getting crashed.

Related-Bug: #1445692
(cherry picked from commit fcde4a30ed26c250c3d177583b5f7af0b97ed6dd)

Change-Id: Ied1723b697b4f397574bfdbc0b1ac8539d09c3a3

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

Reviewed: https://review.opencontrail.org/10643
Committed: http://github.org/Juniper/contrail-web-controller/commit/94c58f858008e0e34eebd87423aee7553c834137
Submitter: Zuul
Branch: master

commit 94c58f858008e0e34eebd87423aee7553c834137
Author: balamurugang <email address hidden>
Date: Fri Apr 24 05:08:26 2015 +0530

In case external gateway for l3 router remains same, but conected VN's are added
or deleted then we need to manually update the route table ref to
"rt_uuid_of_logical_router" in each VN.

In case when external gateway is updated or deleted, then neutron external gateway update
takes care of adding and deleting route table refs for the VN's.

Added error check in the logical router api part..

getting the virtual network using the new api
/api/tenants/config/all-virtual-networks-fields?uuid=
for scaling.

Closes-Bug: #1445692
(cherry picked from commit ecc3bb2cc57ab4e90b1adea9ee8131145d49c4a6)

Updating logical route ref in network's Route table.

In port page
In case of create and edit if the logical router is set for a port.
The logical router reference has to be added manualley in route table of the
network in the format of rt_uuidOfLogicalRouter.
Same way if the router is detached from the port the reference has to be removed
from the corresponding port's network.
Also if the logical router reference is changed the new reference has to be
updated in the network.
All the case are handled.

The api has got exported from logical router api page and reused in ports page.

Related-Bug: #1445692
(cherry picked from commit fcee4b8db0aae42c57c7f0790197b5770ccbc682)

Bug:
In Logical router page of config
At the time of edit When the external gateway is changed to another external
gateway. The nova command fail to set the refference and getting crashed.
Fix:
To resolve the issue the nova command is executed with empty first and then
getting set to the actual value so that nova is not getting crashed.

Related-Bug: #1445692
(cherry picked from commit fcde4a30ed26c250c3d177583b5f7af0b97ed6dd)

Change-Id: Ied1723b697b4f397574bfdbc0b1ac8539d09c3a3

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

Other bug subscribers

Remote bug watches

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