RTC should limit eBGP peers to which VPN routes are advertised

Bug #1582933 reported by Nischal Sheth
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.20
Fix Committed
High
Nischal Sheth
R2.21.x
Fix Committed
High
Nischal Sheth
R2.22.x
Fix Committed
High
Nischal Sheth
R3.0
Fix Committed
High
Nischal Sheth
Trunk
Fix Committed
High
Nischal Sheth

Bug Description

Current implementation of RTC (RFC 4684) advertises VPN routes to all
eBGP peers by default. This includes non-best eBGP peers in multiple
ASes.

Change this to only consider peers in the same neighbor AS as the best
eBGP path for a given RTC route. Note that different RTC routes for
the same RT can and will result in VPN routes being advertised to more
than 1 AS. This is correct behavior.

Nischal Sheth (nsheth)
description: updated
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0

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

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

Review in progress for https://review.opencontrail.org/20337
Submitter: Nischal Sheth (<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/20338
Submitter: Nischal Sheth (<email address hidden>)

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

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

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

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

Nischal Sheth (nsheth)
no longer affects: juniperopenstack/r2.20.x
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/20336
Committed: http://github.org/Juniper/contrail-controller/commit/1c670069a175c22fadd342b5a105b72d130b299e
Submitter: Zuul
Branch: R3.0

commit 1c670069a175c22fadd342b5a105b72d130b299e
Author: Nischal Sheth <email address hidden>
Date: Tue May 17 13:55:15 2016 -0700

Use subset of rtarget eBGP paths when advertising vpn routes

Only consider paths that are learnt from same neighbor AS as
best eBGP path.

Note that different RTC routes for the same RT can and will
result in VPN routes being advertised to more than 1 AS. This
is correct behavior.

Change-Id: I7a08c452c9ee5788fdd615e58a096796752070e0
Closes-Bug: 1582933

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

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

commit d308890ac6cb66a8a6b2f64121a2ecf32b1052bd
Author: Nischal Sheth <email address hidden>
Date: Tue May 17 13:55:15 2016 -0700

Use subset of rtarget eBGP paths when advertising vpn routes

Only consider paths that are learnt from same neighbor AS as
best eBGP path.

Note that different RTC routes for the same RT can and will
result in VPN routes being advertised to more than 1 AS. This
is correct behavior.

Change-Id: I7a08c452c9ee5788fdd615e58a096796752070e0
Closes-Bug: 1582933

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

Review in progress for https://review.opencontrail.org/20339
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/20338
Committed: http://github.org/Juniper/contrail-controller/commit/d0d94c03872fe5096e17139e507620396a486680
Submitter: Zuul
Branch: R2.20

commit d0d94c03872fe5096e17139e507620396a486680
Author: Nischal Sheth <email address hidden>
Date: Tue May 17 13:55:15 2016 -0700

Use subset of rtarget eBGP paths when advertising vpn routes

Only consider paths that are learnt from same neighbor AS as
best eBGP path.

Note that different RTC routes for the same RT can and will
result in VPN routes being advertised to more than 1 AS. This
is correct behavior.

Change-Id: I7a08c452c9ee5788fdd615e58a096796752070e0
Closes-Bug: 1582933

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

Reviewed: https://review.opencontrail.org/20340
Committed: http://github.org/Juniper/contrail-controller/commit/355b4db715086d8c653052ea1ec8bcbd923297e9
Submitter: Zuul
Branch: R2.22.x

commit 355b4db715086d8c653052ea1ec8bcbd923297e9
Author: Nischal Sheth <email address hidden>
Date: Tue May 17 13:55:15 2016 -0700

Use subset of rtarget eBGP paths when advertising vpn routes

Only consider paths that are learnt from same neighbor AS as
best eBGP path.

Note that different RTC routes for the same RT can and will
result in VPN routes being advertised to more than 1 AS. This
is correct behavior.

Change-Id: I7a08c452c9ee5788fdd615e58a096796752070e0
Closes-Bug: 1582933

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

Review in progress for https://review.opencontrail.org/20339
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/20339
Committed: http://github.org/Juniper/contrail-controller/commit/524169697a217e7187c2b376a7a03f82a7433d9f
Submitter: Zuul
Branch: R2.21.x

commit 524169697a217e7187c2b376a7a03f82a7433d9f
Author: Nischal Sheth <email address hidden>
Date: Tue May 17 13:55:15 2016 -0700

Use subset of rtarget eBGP paths when advertising vpn routes

Only consider paths that are learnt from same neighbor AS as
best eBGP path.

Note that different RTC routes for the same RT can and will
result in VPN routes being advertised to more than 1 AS. This
is correct behavior.

Change-Id: I7a08c452c9ee5788fdd615e58a096796752070e0
Closes-Bug: 1582933

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.