PBB EVPN support

Bug #1645092 reported by Hari Prasad Killi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
Fix Committed
Wishlist
Prakash Bailkeri

Bug Description

Tracking bug to implement PBB EVPN in Contrail.

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

Review in progress for https://review.opencontrail.org/26483
Submitter: Prakash Bailkeri (<email address hidden>)

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

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

commit f15a3439c93b9075d6af58cf80fec651a2d74c34
Author: Prakash Bailkeri <email address hidden>
Date: Mon Nov 28 11:17:27 2016 +0530

Schema changes for PBB EVPN

Main changes:
1. MAC learning/aging/mac-move attributes in virtual network/global config
2. PBB configurations on Virtual network
3. bridge domina ifmap identifier and its link to virtual network
4. Mapping VMI to bridge domain based on vlan tag
5. Leaf indication in xmpp enet message
6. Sticky bit indication in xmpp unicast/enet message. This indicates
that address is statically provisioned and can't move.

Change-Id: I0814f9b1c80cbb61bb2f8759ae3a4a7a4d7a9169
Related-bug: #1645092

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

Review in progress for https://review.opencontrail.org/26963
Submitter: Prakash Bailkeri (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/27028
Submitter: Prakash Bailkeri (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/26963
Committed: http://github.org/Juniper/contrail-generateDS/commit/76e64fd3204aff433ad2383258e65e62809ca41c
Submitter: Zuul (<email address hidden>)
Branch: master

commit 76e64fd3204aff433ad2383258e65e62809ca41c
Author: Prakash Bailkeri <email address hidden>
Date: Wed Dec 7 22:26:48 2016 +0530

Support attribute for xmpp messaging

Currently, XML for xmpp message doesn't support attribute declaration.

Example:
Mobility extended community needs two information. sequence number and sticky bit
In the current schema, this needs to be defined as two different element.
<xsd:element name="sequence-number" type="xsd:integer"/>
<xsd:element name="sticky" type="xsd:integer"/>

This will add two xml nodes to xmpp message making it more verbose and bulky
<xmpp-message>
<sequence-number> 1 </sequence-number>
<sticky> False </sticky>
</xmpp-message>

This xmpp message can be represented with one XML node with attributes.
<xmpp-message>
<mobility seqno="2" sticky="true" />
</xmpp-message>

Add support for XML attribute in schema and auto generation of types/parser code

Change-Id: Idbd238ec414f7b18c897c8482cfdff04c0f77876
Related-bug: #1645092

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

Review in progress for https://review.opencontrail.org/27028
Submitter: Prakash Bailkeri (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/27065
Submitter: Prakash Bailkeri (<email address hidden>)

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

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

commit e039251521aafbee673b0de71e289a95e2cce6cb
Author: Prakash Bailkeri <email address hidden>
Date: Thu Dec 8 11:10:11 2016 +0530

Add Mobility info to xmpp enet and inet message

To support mobility extended community, add sequence number and sticky bit to
xmpp route messages.

Xmpp message xml message will carry a tag "mobility" with "seqno" and "sticky"
as attribute.

Example:
<xmpp-message>
<mobility seqno="2" sticky="true" />
</xmpp-message>

Change-Id: Ife4909cff8e2febe349c44b48c5cae7abf0440f9
Related-bug: #1645092

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

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

commit 6743e06d11432bed854bb125e0081ff822d283b7
Author: Prakash Bailkeri <email address hidden>
Date: Thu Dec 8 16:25:22 2016 +0530

Extended community for etree-leaf

Add support for etree extended community
Add sticky bit to MacMobility community
Add tests for etree
Add/Modify tests for MacMobility

Change-Id: I4c167ab12a79e89f3e4bcc4b7998cf20814e2a91
Related-bug: #1645092

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

Review in progress for https://review.opencontrail.org/27313
Submitter: Prakash Bailkeri (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/27375
Submitter: Prakash Bailkeri (<email address hidden>)

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

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

commit cac8301bad38a966108ec754446f84f1b936dfa9
Author: Prakash Bailkeri <email address hidden>
Date: Thu Dec 15 10:36:06 2016 +0530

Schema changes to control insertion of control word

Add boolean config parameter(evpn-control-word) as property to Virtual network.
This flag controls whether or not control word(4-octet with value 0) is inserted
for EVPN packet when sending over MPLS encap.

Change-Id: I41e21b4e36580e1bb75b4be46ca13dab9c29c2ee
Related-bug: #1645092

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

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

commit f15fba4700d970fee588665624e6fb0dd8b445ab
Author: Prakash Bailkeri <email address hidden>
Date: Fri Dec 16 23:24:21 2016 +0530

xmpp message encoding for sticky and leaf flag

Add support for rxing Mobility info and etree-leaf flag
Add support for seding (xmpp_msg_builder) for Mobility and etree-flag
Add tests for tx/rx of mobility info and etree-leaf in xmpp evpn UT

Change-Id: I5b2e5847f17b0686ab7ece2325c9cb795c82f413
Related-bug: #1645092

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

Review in progress for https://review.opencontrail.org/28400
Submitter: Prakash Bailkeri (<email address hidden>)

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

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

commit 37edc22d1b090aa52c438e015a0e2452bc44a419
Author: Prakash Bailkeri <email address hidden>
Date: Fri Feb 3 18:41:46 2017 +0530

PBB EVPN changes

Issues fixed:
1. Avoid generating multiple olist when multiple bridge domains are linked to
b-comp vn
2. Test to validate above fix.
3. Path selection to take into account sticky and etree attribute
4. Sandesh introspect display of etree community
5. Sandesh introspect to display pbb mode on RI(both config and oper data)

Change-Id: I2858af0d1dfc280163da10379331a6bf1b58638a
Related-bug: #1645092

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

Review in progress for https://review.opencontrail.org/28452
Submitter: Prakash Bailkeri (<email address hidden>)

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

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

commit a96dc1e7cbf845dfc0eea764c24d0bde7aa2b7eb
Author: Prakash Bailkeri <email address hidden>
Date: Sun Feb 5 22:20:49 2017 +0530

PBB EVPN changes

Rename BgpAttr method leaf to etree_leaf and sticky to evpn_sticky_mac

Change-Id: I307ad2dc6807a00f0269436ac3db19706804e4de
Related-bug: #1645092

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

Review in progress for https://review.opencontrail.org/28469
Submitter: Prakash Bailkeri (<email address hidden>)

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

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

commit 3cbe6fd5a33b5eb79d9c6669a056a3ec8e7c4663
Author: Prakash Bailkeri <email address hidden>
Date: Mon Feb 6 16:30:15 2017 +0530

PBB EVPN changes

Changes included:
1. Schema changes to make bridge-domain as child of virtua-network

2. Don't allow VMI to refer to different bridge domain for same vlan tag.
e.g. Say, VMI refers to BD1 for vlan tag 0, don't allow this VMI to refer to
another bridge domain(BD2) for vlan tag 0.

3. Add Unit test case for restricting VMI link to single bridge-domain
for a given vlan tag

Change-Id: Ic8599153a4a413b118b7e4968d9988b02aa6c539
Related-bug: 1645092
Closes-Bug: 1653941
Closes-Bug: 1653938

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

Review in progress for https://review.opencontrail.org/28889
Submitter: Prakash Bailkeri (<email address hidden>)

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

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

commit 4440a8314f91cf5ec0ff708ac753485a651c69c2
Author: Prakash Bailkeri <email address hidden>
Date: Fri Feb 17 09:49:40 2017 +0530

Config check for bridge domain creation and VMI linking

1. Allow only one bridge domain to be created in a VN(B-Comp)
2. Allow VMI to Bridge domain link only if VN and Bridge domain belongs
to same virtual network.
3. Add tests cases for validating the above config checks

Related-bug: #1645092

Change-Id: I283479e39e3787e948557d83d432354a49e15b15

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.