Provide customized field selection for ecmp load-balancing

Bug #1498765 reported by Ananth Suryanarayana
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
Fix Committed
Medium
Ananth Suryanarayana

Bug Description

This bug is to track control-node changes to provide customized load-balancing support for opencontrail

Changed in juniperopenstack:
assignee: nobody → Ananth Suryanarayana (anantha-l)
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/14008
Submitter: Ananth Suryanarayana (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/14030
Submitter: Ananth Suryanarayana (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/14008
Submitter: Ananth Suryanarayana (<email address hidden>)

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

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

commit 8bee7e0eb64210bce8d7abbbe34651e06dc176d3
Author: Ananth Suryanarayana <email address hidden>
Date: Mon Oct 5 10:14:03 2015 -0700

Schema changes for customized route's load-balance attribute

o List of packet fields to hash for lad balancing
o Additional actions such as 'source-bias'

Change-Id: I9820b39345707b045c934a2a4ae92ff2511976d0
Partial-Bug: #1498765

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

Review in progress for https://review.opencontrail.org/14030
Submitter: Ananth Suryanarayana (<email address hidden>)

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

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

commit 19bb7bd7806db03242f4bd2f183f19ed353802b6
Author: Ananth Suryanarayana <email address hidden>
Date: Wed Oct 7 01:18:53 2015 -0700

Add support to exchange load-balance bgp extended community

Also addressed review comments from Nischal

o Exchange list of fields to hash upon during ecmp load-balancing
o Fields: L2 Source Address, L2 Dest Address, L3 Source Address, L3 Dest Address,
L4 Protocol, L4 Source Port, L4 Dest Port
o Room for future load-balance fields addition such as interface
o Additional actions field such as source_bias boolean
o Room for future actions such as urpf
o This info is sent to agents via xmpp
o This info is sent to other bgp peers via opaque extended community

o Cleanup hard-coded extended community types with defines

Change-Id: I22371305eb9721ed621979f53382f1fdc495fc1b
Closes-Bug: #1498765

Revision history for this message
Nischal Sheth (nsheth) wrote :

Need to add tests that combine customized load balancing and service chaining.

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

Review in progress for https://review.opencontrail.org/14523
Submitter: Ananth Suryanarayana (<email address hidden>)

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

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

commit bbfb5fcaa1b8f9ac4e428a77e1d369e7b8a752a5
Author: Ananth Suryanarayana <email address hidden>
Date: Mon Oct 19 11:09:49 2015 -0700

Add load-balance attribute for static routes and service chains

During service chain route update, borrow load-balanace attribute
from connected_path if present. If not, borrow from original
path if present.

Static routes automatically get similar treatment. Test cases have
been added to verify the same.

Change-Id: I83ff19e394075dea4dfb7f03b957b6fd5e2bb55e
Closes-Bug: #1498765

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

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

commit f42b6a606353a21cb2dbf29da5e6774cfc6fe21a
Author: Nischal Sheth <email address hidden>
Date: Tue Nov 3 10:42:56 2015 -0800

Minor cleanup for load_balance.[cc|h]

- Fix cpplint nits
- Minor tweak to LoadBalance::ToString

Change-Id: I4bd342c49c939a3fea9263b48b06f69b16836c39
Closes-Bug: 1498765

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

Review in progress for https://review.opencontrail.org/15407
Submitter: Manish Singh (<email address hidden>)

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

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

commit 98f10a8a5167298fc79077960f400bf34735cdfe
Author: Manish Singh <email address hidden>
Date: Wed Nov 25 08:03:36 2015 +0530

Schema changes of Ecmp load balancing configuration.

This change enables user to choose fields to do load balance in ecmp nexthops.
These criterias can be specified at interface/VN/global level.

Change-Id: Ibd349bc156deaff77af4b126f953077f8344d339
Partial-bug: 1498765

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

Review in progress for https://review.opencontrail.org/16283
Submitter: Manish Singh (<email address hidden>)

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

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

commit f6dd3dc4750ea0fa18c3e0989d7519de05c19511
Author: Manish Singh <email address hidden>
Date: Wed Jan 13 16:07:24 2016 +0530

ECMP load balance configuration agent changes.

ECMP load balance parameters can be configured at global, vn and interface level.
Priority (highest to lowest)- Interface, VN, Global.
This config is stored in path. When path is exported to controller, fields in
load-balance-fields are populated using the same. Agent currently provides field
based hashing only. So every controller message will have load-balance-type set
to field-hash. Similarly reflected routes from CN will be used to extract fields
to be used for hashing and populated in respective path.
Agent by default will use all fields.

Closes-bug: 1498765

Conflicts:
 src/vnsw/agent/controller/controller_peer.cc
 src/vnsw/agent/openstack/instance_service_server.cc
 src/vnsw/agent/oper/inet_unicast_route.cc
 src/vnsw/agent/oper/metadata_ip.cc
 src/vnsw/agent/oper/vm_interface.cc
 src/vnsw/agent/test/test_util.cc

Change-Id: I652f65b5b6b56fe20e26a0a0bb7cdf6799b98811

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.