Implement EVPN aliasing support

Bug #1731351 reported by Nischal Sheth
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R4.1
Fix Committed
Wishlist
Nischal Sheth
Trunk
Fix Committed
Wishlist
Nischal Sheth

Bug Description

Implement EVPN aliasing support.

Nischal Sheth (nsheth)
Changed in juniperopenstack:
milestone: none → r5.0.0
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

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

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

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

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

Review in progress for https://review.opencontrail.org/37375
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/37375
Committed: http://github.com/Juniper/contrail-controller/commit/240c7b44e6f6249ebed620f08150e7f63e9b2408
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit 240c7b44e6f6249ebed620f08150e7f63e9b2408
Author: Nischal Sheth <email address hidden>
Date: Sat Nov 4 23:26:34 2017 -0700

Implement aliasing for EVPN with VXLAN encapsulation

Change-Id: I1b59306888d1b2db6919ee65e441b277a26833d7
Partial-Bug: 1731351

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

Review in progress for https://review.opencontrail.org/37444
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/37372
Committed: http://github.com/Juniper/contrail-controller/commit/d64e1e83b6f052edde6b71b74fc1766fc81d94a1
Submitter: Zuul (<email address hidden>)
Branch: master

commit d64e1e83b6f052edde6b71b74fc1766fc81d94a1
Author: Nischal Sheth <email address hidden>
Date: Sat Nov 4 23:26:34 2017 -0700

Implement aliasing for EVPN with VXLAN encapsulation

Change-Id: I1b59306888d1b2db6919ee65e441b277a26833d7
Partial-Bug: 1731351

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

Review in progress for https://review.opencontrail.org/37478
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/37444
Committed: http://github.com/Juniper/contrail-controller/commit/4e80292b70248511bd673224d57bfc6d9345aed4
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit 4e80292b70248511bd673224d57bfc6d9345aed4
Author: Nischal Sheth <email address hidden>
Date: Sun Nov 12 16:55:54 2017 -0800

Address review comments for EVPN aliasing

Change-Id: Ic8a6625e1e6df6aef94641f483c1d221df89dbc6
Partial-Bug: 1731351

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

Review in progress for https://review.opencontrail.org/37514
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/37514
Committed: http://github.com/Juniper/contrail-controller/commit/3787ea1abea21e992b36690dd9f151d03858192f
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit 3787ea1abea21e992b36690dd9f151d03858192f
Author: Nischal Sheth <email address hidden>
Date: Tue Nov 14 15:28:53 2017 -0800

Don't ignore secondary paths when looking for path by nexthop

Change-Id: I244209280fb23d1e0d27e66fd5ac8003baf9247c
Partial-Bug: 1731351

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

Reviewed: https://review.opencontrail.org/37478
Committed: http://github.com/Juniper/contrail-controller/commit/dc888200aa117ac32e73e41182b13ca869fcda7a
Submitter: Zuul (<email address hidden>)
Branch: master

commit dc888200aa117ac32e73e41182b13ca869fcda7a
Author: Nischal Sheth <email address hidden>
Date: Sun Nov 12 16:55:54 2017 -0800

Address review comments for EVPN aliasing

Change-Id: Ic8a6625e1e6df6aef94641f483c1d221df89dbc6
Partial-Bug: 1731351

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

Review in progress for https://review.opencontrail.org/37541
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/37541
Committed: http://github.com/Juniper/contrail-controller/commit/00cb24f67b2827c48bf694fc5f9beb98dfa92d23
Submitter: Zuul (<email address hidden>)
Branch: master

commit 00cb24f67b2827c48bf694fc5f9beb98dfa92d23
Author: Nischal Sheth <email address hidden>
Date: Tue Nov 14 15:28:53 2017 -0800

Don't ignore secondary paths when looking for path by nexthop

Change-Id: I244209280fb23d1e0d27e66fd5ac8003baf9247c
Partial-Bug: 1731351
(cherry picked from commit 3787ea1abea21e992b36690dd9f151d03858192f)

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

Review in progress for https://review.opencontrail.org/37637
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/37638
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/37637
Committed: http://github.com/Juniper/contrail-controller/commit/1350e18699190d9bbe85affd02bca3d30d186e77
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit 1350e18699190d9bbe85affd02bca3d30d186e77
Author: Nischal Sheth <email address hidden>
Date: Fri Nov 17 11:58:07 2017 -0800

Allow EvpnSegment processing for EvpnManagers to happen in parallel

Change-Id: I2ea44e003567975d803e4d6ead23f7f8c494a60b
Partial-Bug: 1731351

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

Review in progress for https://review.opencontrail.org/37671
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/37671
Committed: http://github.com/Juniper/contrail-controller/commit/8b878f1de7af14c2af05f717505729f253c58fdb
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit 8b878f1de7af14c2af05f717505729f253c58fdb
Author: Nischal Sheth <email address hidden>
Date: Sun Nov 19 22:51:04 2017 -0800

Revert 'Allow EvpnSegment processing for EvpnManagers to happen in parallel'

Change-Id: I47950c3695887b85a8ff9f7a4ff139e2ee2d6340
Partial-Bug: 1731351

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

Review in progress for https://review.opencontrail.org/37817
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/37818
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/37817
Committed: http://github.com/Juniper/contrail-controller/commit/c008d6da711d833b59333888ac67f66a64e00bbd
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit c008d6da711d833b59333888ac67f66a64e00bbd
Author: Nischal Sheth <email address hidden>
Date: Wed Nov 22 17:03:03 2017 -0800

Tweak scheduler policy for bgp::EvpnSegment task

Tweak scheduler policy for bgp::EvpnSegment task to present it from
running concurrently with any other task that manipulates the DB.

Change-Id: I91f5836a7bcf8a5ef6cbfb15fe306cb04a05fd23
Partial-Bug: 1731351

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

Reviewed: https://review.opencontrail.org/37818
Committed: http://github.com/Juniper/contrail-controller/commit/b3b94366fde3345017354c89adbf4c503850af51
Submitter: Zuul (<email address hidden>)
Branch: master

commit b3b94366fde3345017354c89adbf4c503850af51
Author: Nischal Sheth <email address hidden>
Date: Wed Nov 22 17:03:03 2017 -0800

Tweak scheduler policy for bgp::EvpnSegment task

Tweak scheduler policy for bgp::EvpnSegment task to present it from
running concurrently with any other task that manipulates the DB.

Change-Id: I91f5836a7bcf8a5ef6cbfb15fe306cb04a05fd23
Partial-Bug: 1731351

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

Review in progress for https://review.opencontrail.org/37853
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/37854
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/37853
Committed: http://github.com/Juniper/contrail-controller/commit/052c02c2b8dfdde99e6adb08a70ff3a7ca8743d0
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit 052c02c2b8dfdde99e6adb08a70ff3a7ca8743d0
Author: Nischal Sheth <email address hidden>
Date: Sun Nov 19 21:19:33 2017 -0800

Wrap up EVPN aliasing implementation

o Use per-ESI AD route peer and attributes to add aliased paths instead
of using the MAC route BgpPath peer and attributes. This will make it
easier to figure out which AD routes triggered the addition of aliased
paths.
o Remove route targets when importing AD routes into VRF. This ensures
that the aliased paths do not have a large number of route targets.
Note that we expect per ESI AD routes to have a large number of route
targets.
o Simplify EvpnSegment::RemotePe constructor and usage
o Make bgp_evpn_aliasing_test more robust by disabling/enabling various
TaskTriggers from the bgp:Config task instead of the main thread.

Change-Id: Ic019bda64f79c9dd52415f74d2b9fc856c26e431
Closes-Bug: 1731351

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

Reviewed: https://review.opencontrail.org/37854
Committed: http://github.com/Juniper/contrail-controller/commit/e0b47a612c12208582c32e0c0653cbbe82306f84
Submitter: Zuul (<email address hidden>)
Branch: master

commit e0b47a612c12208582c32e0c0653cbbe82306f84
Author: Nischal Sheth <email address hidden>
Date: Sun Nov 19 21:19:33 2017 -0800

Wrap up EVPN aliasing implementation

o Use per-ESI AD route peer and attributes to add aliased paths instead
of using the MAC route BgpPath peer and attributes. This will make it
easier to figure out which AD routes triggered the addition of aliased
paths.
o Remove route targets when importing AD routes into VRF. This ensures
that the aliased paths do not have a large number of route targets.
Note that we expect per ESI AD routes to have a large number of route
targets.
o Simplify EvpnSegment::RemotePe constructor and usage
o Make bgp_evpn_aliasing_test more robust by disabling/enabling various
TaskTriggers from the bgp:Config task instead of the main thread.

Change-Id: Ic019bda64f79c9dd52415f74d2b9fc856c26e431
Closes-Bug: 1731351

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

Review in progress for https://review.opencontrail.org/37957
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/37958
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/37957
Committed: http://github.com/Juniper/contrail-controller/commit/444f087109c577d611d3a12c1338acba2a7dfb82
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit 444f087109c577d611d3a12c1338acba2a7dfb82
Author: Nischal Sheth <email address hidden>
Date: Tue Nov 28 16:28:28 2017 -0800

Make aliased path less preferred

Change-Id: I40f7d8212cb888c45a3281abfa1d708e5fe45559
Closes-Bug: 1731351

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

Reviewed: https://review.opencontrail.org/37958
Committed: http://github.com/Juniper/contrail-controller/commit/29fc19ee93920ada5ccde9c3deb3730b8006fad8
Submitter: Zuul (<email address hidden>)
Branch: master

commit 29fc19ee93920ada5ccde9c3deb3730b8006fad8
Author: Nischal Sheth <email address hidden>
Date: Tue Nov 28 16:28:28 2017 -0800

Make aliased path less preferred

Change-Id: I40f7d8212cb888c45a3281abfa1d708e5fe45559
Closes-Bug: 1731351

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.