Provide per bgp peer local-as support

Bug #1755768 reported by Ananth Suryanarayana
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.2
Won't Fix
High
Ananth Suryanarayana
R4.0
Won't Fix
High
Ananth Suryanarayana
R4.1
Fix Committed
High
Manoj
R5.0
Fix Committed
High
Manoj
Trunk
Fix Committed
High
Manoj

Bug Description

Some bgpaas deployments need different peer AS number for each bgpaas session (on the control-node) side. Hence, provide a local-as support on a per bgp peer basis

summary: - Privide per bgp peer local-as support
+ Provide per bgp peer local-as support
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/40629
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/40630
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/40629
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/40630
Submitter: Ananth Suryanarayana (<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/40807
Submitter: Ananth Suryanarayana (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/40808
Submitter: Ananth Suryanarayana (<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/40809
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/40629
Committed: http://github.com/Juniper/contrail-api-client/commit/c62d3a29cac0172f467da88ee5179d7fa6c56044
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit c62d3a29cac0172f467da88ee5179d7fa6c56044
Author: Ananth Suryanarayana <email address hidden>
Date: Wed Mar 14 03:36:53 2018 -0700

Provide per bgp peer local-as support

Some bgpaas deployments need different peer AS number for each bgpaas session
(on the control-node) side. Hence, provide a local-as support on a per bgp peer
basis. Add local-autonomous-system field to bgp session attributes in the
schema

Change-Id: I71a5250f50cf3cdf1090644f9da9883183632111
Partial-Bug: 1755768

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

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

commit 892dfeca432c50e64752ce6981ff34eff5d0f735
Author: Ananth Suryanarayana <email address hidden>
Date: Mon Mar 19 05:19:28 2018 -0700

Provide local-as configure-ability for each bgp session

Some bgpaas deployments need different peer AS number for each bgpaas session
(on the control-node) side. Hence, provide a local-as support on a per bgp peer
basis

o local_as_ is already of BgpPeer. It was always populated with either the
global asn or (per control-node) local asn. With this change, if local-as
is configured on a per peering session, then that local-as takes precendence.
o Add unit tests to test this per per specific local as
o Parameterize bgp_bgpaas_test to test with per peer local-as configured or
otherwise
o Tweak RibOut policy to create sepearte ribouts if local-as field differs
o During as-path prepend, either global local-as or per peer local-as is
prepended for ebgp sessions.
o AS Override shall now override with per peer local-as if configured,
global local-as otherwise

Change-Id: I2d1e30820195c2eacf392317c235ac40ad7ce027
Partial-Bug: 1755768

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

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

commit 31a8af589dc29b9f5590a43018e698f20017cb2f
Author: Ananth Suryanarayana <email address hidden>
Date: Wed Mar 14 03:39:23 2018 -0700

Provide local-as configure-ability for each bgp session

Some bgpaas deployments need different peer AS number for each bgpaas session
(on the control-node) side. Hence, provide a local-as support on a per bgp peer
basis

o local_as_ is already of BgpPeer. It was always populated with either the
global asn or (per control-node) local asn. With this change, if local-as
is configured on a per peering session, then that local-as takes precendence.
o Add unit tests to test this per per specific local as
o Parameterize bgp_bgpaas_test to test with per peer local-as configured or
otherwise
o Tweak RibOut policy to create sepearte ribouts if local-as field differs
o During as-path prepend, either global local-as or per peer local-as is
prepended for ebgp sessions.
o AS Override shall now override with per peer local-as if configured,
global local-as otherwise

Change-Id: I2d1e30820195c2eacf392317c235ac40ad7ce027
Depends-On: I71a5250f50cf3cdf1090644f9da9883183632111
Partial-Bug: 1755768

Revision history for this message
Ananth Suryanarayana (anantha-l) wrote :

Hi Rudra, Can you please reassign this to the (new) UI team, to add this new local-as option to bgpaas peers ? I believe UI can make necessary vnc-api calls to make the local-as configuration take effect

Revision history for this message
Ananth Suryanarayana (anantha-l) wrote :

Can you please add local-as option to bgpaas peers configuration in the UI.

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

Review in progress for https://review.opencontrail.org/41423
Submitter: Manoj (<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/41424
Submitter: Manoj (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/41426
Submitter: Manoj (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/41424
Committed: http://github.com/Juniper/contrail-web-controller/commit/7b2642177b8a435608e85f24794b6eb684035295
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit 7b2642177b8a435608e85f24794b6eb684035295
Author: Manoj Naik <email address hidden>
Date: Wed Apr 4 14:18:59 2018 -0700

Added Local ASN configuration to BGP peer

Added Local AS field to BGP peer configuration.

Change-Id: I19529df9da0b84de432e78d2c2b2f3c892d23b46
Closes-Bug: #1755768

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

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

commit d018c44d8c192eb2ef6eed5b9d519c824afb9d56
Author: Manoj Naik <email address hidden>
Date: Wed Apr 4 14:18:59 2018 -0700

Added Local ASN configuration to BGP peer

Added Local AS field to BGP peer configuration.

Change-Id: I19529df9da0b84de432e78d2c2b2f3c892d23b46
Closes-Bug: #1755768

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

Reviewed: https://review.opencontrail.org/41426
Committed: http://github.com/Juniper/contrail-web-controller/commit/fb9cf0f1567c5d1c5e64b4bdf6a1c4fe1ba973ac
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0

commit fb9cf0f1567c5d1c5e64b4bdf6a1c4fe1ba973ac
Author: Manoj Naik <email address hidden>
Date: Wed Apr 4 14:18:59 2018 -0700

Added Local ASN configuration to BGP peer

Added Local AS field to BGP peer configuration.

Change-Id: I19529df9da0b84de432e78d2c2b2f3c892d23b46
Closes-Bug: #1755768

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

Review in progress for https://review.opencontrail.org/43470
Submitter: Manoj (<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/43471
Submitter: Manoj (<email address hidden>)

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

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

commit b3c05364abb819b05d9657b0207e04f3961ede34
Author: Manoj Naik <email address hidden>
Date: Fri Jun 1 11:40:46 2018 -0700

Added the Local ASN field for BGPaaS page

Added Local ASN configuration for BGPaaS page.

Change-Id: I33f7393d24f39eab77d924b4a5da24e121f66423
Closes-Bug: #1755768

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

Reviewed: https://review.opencontrail.org/43471
Committed: http://github.com/Juniper/contrail-web-controller/commit/71f066edf5367c3a542a8a6bf851fa08eea45c19
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit 71f066edf5367c3a542a8a6bf851fa08eea45c19
Author: Manoj Naik <email address hidden>
Date: Fri Jun 1 11:40:46 2018 -0700

Added the Local ASN field for BGPaaS page

Added Local ASN configuration for BGPaaS page.

Change-Id: I33f7393d24f39eab77d924b4a5da24e121f66423
Closes-Bug: #1755768

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.