ECMP logic should compare AS path length

Bug #1582452 reported by vivekananda shenoy
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.0
Fix Committed
High
Nischal Sheth
Trunk
Fix Committed
High
Nischal Sheth

Bug Description

Version 3.0.2.0 build 25

If CN receives multiple BGP path for the same IP prefix from another Contrail controller or from the MX the bgp best path check is done by comparing local-preference and in case if one of the path has shorter AS-PATH length compared to the other one both are treated equal.

VES topo is attached. Cluster1 is getting 3 paths for IP address of a VM in cluster 2 and all 3 paths are installed in the agent routing table although the direct EBGP path from cluster 2 has AS-PATH-LEN of 1. This is creating issues w.r.t ARP resolution for the IP address of VM in cluster 2, as the ARP is getting resolved to vhost0 MAC address of the compute node in cluster 2. I am opening a separate PR for that.

Revision history for this message
vivekananda shenoy (vshenoy83) wrote :
description: updated
Nischal Sheth (nsheth)
description: updated
Changed in juniperopenstack:
importance: Undecided → High
assignee: nobody → Nischal Sheth (nsheth)
Nischal Sheth (nsheth)
summary: - 3.0 Controller : There has to be check against the AS path length when
- comparing ECMP routes in the case of federated clusters
+ ECMP logic should compare AS path length in case of federated clusters
summary: - ECMP logic should compare AS path length in case of federated clusters
+ ECMP logic should compare AS path length
description: updated
tags: added: contrail-control
tags: added: blocker
Nischal Sheth (nsheth)
information type: Proprietary → Public
tags: removed: blocker
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

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

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

Review in progress for https://review.opencontrail.org/20578
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/20578
Committed: http://github.org/Juniper/contrail-controller/commit/6ccf3c411fa416a8d59bf4ebae6a9ef25d92c97c
Submitter: Zuul
Branch: R3.0

commit 6ccf3c411fa416a8d59bf4ebae6a9ef25d92c97c
Author: Nischal Sheth <email address hidden>
Date: Tue May 24 13:18:48 2016 -0700

Include AS path length check in ECMP decision

Note that this changes ecmp behavior from older releases, but this is
deemed to be fine since the new behavior is what most users will expect
by default.

A new knob/option to ignore AS path length can be added if/when needed.

Change-Id: I394f0a683d1ef1fa3db5bd983829acf4aab2c684
Closes-Bug: 1582452

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

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

commit c5daffae842488d61c7fd2bb310b11245754986d
Author: Nischal Sheth <email address hidden>
Date: Tue May 24 13:18:48 2016 -0700

Include AS path length check in ECMP decision

Note that this changes ecmp behavior from older releases, but this is
deemed to be fine since the new behavior is what most users will expect
by default.

A new knob/option to ignore AS path length can be added if/when needed.

Change-Id: I394f0a683d1ef1fa3db5bd983829acf4aab2c684
Closes-Bug: 1582452

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.