Implement Route Table with ip address next hop

Bug #1541087 reported by Sachin Bansal
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.21.x
Won't Fix
High
Suresh Balineni
Trunk
Fix Committed
High
Suresh Balineni

Bug Description

Currently, we have an implementation of route tables where each route can have a service instance as a next hop. We need to be able to specify an ip address as next hop. For this, I propose to change the next-hop-type field in RouteType to an enum with two possible values: 'service-instance' and 'ip-address'. If the field is not set, we will treat it same as 'service-instance'. If it is set to 'ip-address', the next hop must be a valid ip address. We will then create a static-route in the primary routing instance of that network with the same prefix and next hop and no route targets.

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

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

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

Review in progress for https://review.opencontrail.org/16833
Submitter: Biswajit Mandal (<email address hidden>)

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

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

commit 0159b96d9cc6a4e1b216d2b1851a847df6bca0f5
Author: Sachin Bansal <email address hidden>
Date: Wed Feb 3 00:17:06 2016 -0800

Add next hop type to schema

Change-Id: I42f46a53561a6c42b2e84db14bca49d172279591
Related-Bug: 1541087

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

Reviewed: https://review.opencontrail.org/16833
Committed: http://github.org/Juniper/contrail-web-controller/commit/79204d63e8a48df3f351715cb6f8fc7e7f65cf96
Submitter: Zuul
Branch: master

commit 79204d63e8a48df3f351715cb6f8fc7e7f65cf96
Author: Biswajit Mandal <email address hidden>
Date: Wed Feb 3 22:35:50 2016 +0530

Related-Bug: #1541087
Adding next-hop-type as service-instance or ip as per new schema change in route
table

Change-Id: I469cc415bc4c93cc95976a3eb5dd15a5bf236d1a

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

Review in progress for https://review.opencontrail.org/16999
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/16999
Committed: http://github.org/Juniper/contrail-controller/commit/eb6d0b8893bfb6f52cbea89f94bb9e4855d47524
Submitter: Zuul
Branch: master

commit eb6d0b8893bfb6f52cbea89f94bb9e4855d47524
Author: sbalineni <email address hidden>
Date: Mon Feb 8 16:26:59 2016 -0800

ST: Support for NextHop Static IP Routes in Route Tables

Feature implementaion with UT

Closes-Bug: #1541087

Change-Id: I8ee0eb466b8358aa01716eec638e7c80e10546c1

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

Review in progress for https://review.opencontrail.org/17183
Submitter: Nischal Sheth (<email address hidden>)

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

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

commit 957cfc47eb15fef0ae95232ade63e9fb79d2490a
Author: Nischal Sheth <email address hidden>
Date: Wed Feb 10 16:57:58 2016 -0800

Handle static routes in VN's default routing instance

Highlights:

- Fix bug wherein an empty route target list for a static route caused
it to not have any tunnel encap, security groups etc.
- Set the OriginVn for static routes if the route target list is empty
- Update all static routes for a routing instance when vn index changes
- Add unit and integration tests

Change-Id: I658d3be3adaf93f3bb8b67a612869e1c21ae4f9b
Closes-Bug: 1541087

Nischal Sheth (nsheth)
information type: Proprietary → Public
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.21.x

Review in progress for https://review.opencontrail.org/22955
Submitter: Jean-Philippe Braun (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/22956
Submitter: Jean-Philippe Braun (<email address hidden>)

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.