Agent ignoring LP values other than 100/200

Bug #1676495 reported by amit surana
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.0
New
Undecided
Unassigned
R3.1
New
Undecided
Unassigned
R3.2
Fix Committed
Medium
Naveen N
R4.0
Fix Committed
Medium
Naveen N
Trunk
Fix Committed
Medium
Naveen N

Bug Description

vRouter agent only accepts local-preference values of 100/200. Attempt to set LP to any other value results in agent re-setting LP to 100.

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

Review in progress for https://review.opencontrail.org/31430
Submitter: Nabeel Asim (<email address hidden>)

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

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

commit d81d98cc8418ea32ea2bc87fd3a43472b38bd4d5
Author: Nabeel Asim <email address hidden>
Date: Fri May 12 00:37:30 2017 +0500

Support full range of local preference

Contrail controller should be able to provide active backup
functionality with multiple backup options to choose for a
path, ordered based on priority without forming an ECMP.

Virtual machine interface already supports local preference
configuration, but however it internally only allow
configuration of 200 everything else is overridden by value
100. This limits the scope to be able to provide multiple
backup paths to only one.

Converting enum to uint32_t to hold full range of configured
local preference to support multiple backup paths ordered
using local preference.

Change-Id: I549df3631e4f47bf43f4118ee799c74b53bb3a19
Closes-Bug: #1676495

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

Review in progress for https://review.opencontrail.org/31734
Submitter: Nabeel Asim (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/31735
Submitter: Nabeel Asim (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/31735
Committed: http://github.com/Juniper/contrail-controller/commit/fbadfd8e3bfe60ae175a21dab9c7a8941d3d923f
Submitter: Zuul (<email address hidden>)
Branch: R3.2

commit fbadfd8e3bfe60ae175a21dab9c7a8941d3d923f
Author: Nabeel Asim <email address hidden>
Date: Wed May 17 19:00:27 2017 +0500

Support full range of local preference

Contrail controller should be able to provide active backup
functionality with multiple backup options to choose for a
path, ordered based on priority without forming an ECMP.

Virtual machine interface already supports local preference
configuration, but however it internally only allow
configuration of 200 everything else is overridden by value
100. This limits the scope to be able to provide multiple
backup paths to only one.

Converting enum to uint32_t to hold full range of configured
local preference to support multiple backup paths ordered
using local preference.

Change-Id: I549df3631e4f47bf43f4118ee799c74b53bb3a19
Closes-Bug: #1676495

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

Reviewed: https://review.opencontrail.org/31734
Committed: http://github.com/Juniper/contrail-controller/commit/18ff7724befb38174ca3fe8d1bc845fec4b8885a
Submitter: Zuul (<email address hidden>)
Branch: R4.0

commit 18ff7724befb38174ca3fe8d1bc845fec4b8885a
Author: Nabeel Asim <email address hidden>
Date: Fri May 12 00:37:30 2017 +0500

Support full range of local preference

Contrail controller should be able to provide active backup
functionality with multiple backup options to choose for a
path, ordered based on priority without forming an ECMP.

Virtual machine interface already supports local preference
configuration, but however it internally only allow
configuration of 200 everything else is overridden by value
100. This limits the scope to be able to provide multiple
backup paths to only one.

Converting enum to uint32_t to hold full range of configured
local preference to support multiple backup paths ordered
using local preference.

Change-Id: I549df3631e4f47bf43f4118ee799c74b53bb3a19
Closes-Bug: #1676495

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.