For snat static routes, use logical router's route target instead of using each client VN's

Bug #1565111 reported by Sachin Bansal
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
Fix Committed
Medium
Suresh Balineni

Bug Description

Today, when we create an external gateway for a logical router, svc-monitor creates a new route table with default route pointing to the snat service instance and links it to each of the private networks. Then schema transformer creates a static route in the internal left VN of the service instance with route targets taken from each of the client VNs.

Because of this implementation, svc-monitor has to watch networks being added to logical router and update links to the route table accordingly. Also, schema transformer then has to update the static route with the new set of route targets.

The new proposal is to create a link between logical router and route table. Schema transformer will, then create static routes with the route target only of route targets linked to the logical router. Since all client VNs already import logical router's route targets, they will still get the routes without the additional churn in the config.

Additionally, when svc-monitor comes up, it should delink the route tables from client VNs and link it to the logical-router to keep things consistent.

Tags: config snat
Nischal Sheth (nsheth)
information type: Proprietary → Public
tags: added: snat
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/19084
Submitter: Suresh Balineni (<email address hidden>)

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

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

commit 569a9bb85c5297ed9029a2e740eff2710787e333
Author: Suresh Balineni <email address hidden>
Date: Tue Apr 5 23:40:33 2016 +0000

ST: Schema changes for using logical router's route target instead of using each client VN's

Schema changes: For snat static routes, use logical router's route target instead of using each client VN's

Change-Id: I8f9dc5cdaf5891e0c5b932509707ab185956fc62
Partial-Bug: #1565111

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

Review in progress for https://review.opencontrail.org/19571
Submitter: Suresh Balineni (<email address hidden>)

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

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

commit a66d6c2e41d47df2030110208b67d7bd8058750f
Author: Suresh Balineni <email address hidden>
Date: Sat Apr 23 00:46:45 2016 +0000

ST/SM: For snat static routes, use logical router's route target instead of using each client VN's

Change-Id: I5e856eb387cf1a2029b02ce8cdd710e86b02fdd9
Closes-Bug: #1565111

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

Review in progress for https://review.opencontrail.org/19977
Submitter: Suresh Balineni (<email address hidden>)

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

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

commit f68b83a615271990e59d2021e281b421c04e3eb6
Author: Suresh Balineni <email address hidden>
Date: Sat May 7 00:07:39 2016 +0000

ST/SM: For snat static routes, use logical router's route target instead of using each client VN's
Closes-Bug: #1565111

Change-Id: Iae8df2c71cf1c8a748093f40a1afc636f3b076d2

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

Review in progress for https://review.opencontrail.org/20467
Submitter: Suresh Balineni (<email address hidden>)

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

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

commit 9d6b146983c7900c5019e95f876f9a0620f401d1
Author: Suresh Balineni <email address hidden>
Date: Fri May 20 18:13:34 2016 +0000

[ST/SM]: Upgrade support for new implementation of SNAT static routes config generation

- prior to 3.0, SNAT snat static routes are generated from private VNs route-table links
- post 3.0, SNAT static routes are configured using logical router - route table association
- The fixes in this review will take care of upgrading <=3.0 DB to new scheme.
- UT code also added.

Change-Id: I5267f366f35935e83088dd25999ffc21efd145ef
Closes-Bug: #1565111

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.