Comment 44 for bug 1445692

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