Juniper Openstack

[EVPN VXLAN] Multi Homing : DM is overriding ESI value when multiple ESI configured

Bug #1732327 reported by chhandak
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R4.0
Fix Committed
Critical
Suresh Balineni
Juniper Openstack r4.0.3.0
R4.1
Fix Committed
Critical
Suresh Balineni
Juniper Openstack r4.1.0.0-fcs "r4.1"
Trunk
Fix Committed
Critical
Suresh Balineni
Juniper Openstack r5.0.0

Bug Description

When we have 2 different ESI value configured for 2 different physical devices, DM should configure those ESI value for the respective physical device and create 2 different LAG interface. Instead of the when the second physcial device is getting configured DM is overriding the first
one.

As per API server

physical-interface: {
display_name: "xe-0/0/46",
uuid: "6247e820-71fb-4716-bdf7-c0107994a5cf",
ethernet_segment_identifier: "00:11:22:33:44:55:66:77:88:11",

fq_name: [
"default-global-system-config",
"5b11-qfx2",
"xe-0/0/46"
],

physical-interface: {
display_name: "xe-0/0/1",
uuid: "930641e7-701f-4170-9a3c-64082d518c6b",
ethernet_segment_identifier: "00:11:22:33:44:55:66:77:88:99",
parent_uuid: "52db4a55-3ea8-44e8-883a-f94dafed7bdd",

fq_name: [
"default-global-system-config",
"5b11-qfx2",
"xe-0/0/46"
],

DM Config
-----------
root@5b11-qfx2# run show configuration groups __contrail__ interfaces | display set
set groups __contrail__ interfaces lo0 unit 0 family inet address 172.16.2.1/32 primary
set groups __contrail__ interfaces lo0 unit 0 family inet address 172.16.2.1/32 preferred
set groups __contrail__ interfaces ae127 flexible-vlan-tagging
set groups __contrail__ interfaces ae127 encapsulation extended-vlan-bridge
set groups __contrail__ interfaces ae127 esi 00:11:22:33:44:55:66:77:88:99
set groups __contrail__ interfaces ae127 esi all-active
set groups __contrail__ interfaces ae127 aggregated-ether-options lacp active
set groups __contrail__ interfaces ae127 aggregated-ether-options lacp system-priority 100
set groups __contrail__ interfaces ae127 aggregated-ether-options lacp system-id 44:55:66:77:88:99
set groups __contrail__ interfaces ae127 aggregated-ether-options lacp admin-key 1
set groups __contrail__ interfaces ae127 unit 100 vlan-id 100
set groups __contrail__ interfaces xe-0/0/46 gigether-options 802.3ad ae127
set groups __contrail__ interfaces xe-0/0/1 gigether-options 802.3ad ae127

Created a Single LAG device with last updated ESI value

chhandak (chhandak)
tags: added: blocker
Changed in juniperopenstack:
importance: Undecided → Critical
assignee: nobody → Suresh Balineni (sbalineni)
milestone: none → r4.1.0.0-fcs
information type: Proprietary → Public
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R4.1

Review in progress for https://review.opencontrail.org/37548
Submitter: Suresh Balineni (<email address hidden>)

Sachin Bansal (sbansal)
Changed in juniperopenstack:
milestone: r4.1.0.0-fcs → r5.0.0
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

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

commit 563def507677af33f54292d299d8354b5ec79613
Author: sbalineni <email address hidden>
Date: Wed Nov 15 12:56:50 2017 -0800

[DM]: QFX5k - ae allocation: reserve/free right index values

reserve/free right indexes, in the case of reverse allocation of bitarray

generated config (ae127, ae126):

set groups __contrail__ interfaces ae127 flexible-vlan-tagging
set groups __contrail__ interfaces ae127 encapsulation extended-vlan-bridge
set groups __contrail__ interfaces ae127 esi 00:11:22:33:44:55:66:77:88:11
set groups __contrail__ interfaces ae127 esi all-active
set groups __contrail__ interfaces ae127 aggregated-ether-options lacp active
set groups __contrail__ interfaces ae127 aggregated-ether-options lacp system-priority 100
set groups __contrail__ interfaces ae127 aggregated-ether-options lacp system-id 44:55:66:77:88:11
set groups __contrail__ interfaces ae127 aggregated-ether-options lacp admin-key 1
set groups __contrail__ interfaces ae127 unit 100 vlan-id 100
set groups __contrail__ interfaces xe-0/0/46 gigether-options 802.3ad ae127
set groups __contrail__ interfaces ae126 flexible-vlan-tagging
set groups __contrail__ interfaces ae126 encapsulation extended-vlan-bridge
set groups __contrail__ interfaces ae126 esi 00:11:22:33:44:55:66:77:88:99
set groups __contrail__ interfaces ae126 esi all-active
set groups __contrail__ interfaces ae126 aggregated-ether-options lacp active
set groups __contrail__ interfaces ae126 aggregated-ether-options lacp system-priority 100
set groups __contrail__ interfaces ae126 aggregated-ether-options lacp system-id 44:55:66:77:88:99
set groups __contrail__ interfaces ae126 aggregated-ether-options lacp admin-key 1
set groups __contrail__ interfaces ae126 unit 100 vlan-id 100
set groups __contrail__ interfaces xe-0/0/1 gigether-options 802.3ad ae126

Closes-Bug: #1732327

Change-Id: I7a1e6dab96af21f24c840165747a84b822a93061

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

Review in progress for https://review.opencontrail.org/37631
Submitter: Suresh Balineni (<email address hidden>)

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

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

commit a46bbf7fd202d9204c428a695a833dfdc3780fbd
Author: sbalineni <email address hidden>
Date: Wed Nov 15 12:56:50 2017 -0800

[DM]: QFX5k - ae allocation: reserve/free right index values

reserve/free right indexes, in the case of reverse allocation of bitarray

generated config (ae127, ae126):

set groups __contrail__ interfaces ae127 flexible-vlan-tagging
set groups __contrail__ interfaces ae127 encapsulation extended-vlan-bridge
set groups __contrail__ interfaces ae127 esi 00:11:22:33:44:55:66:77:88:11
set groups __contrail__ interfaces ae127 esi all-active
set groups __contrail__ interfaces ae127 aggregated-ether-options lacp active
set groups __contrail__ interfaces ae127 aggregated-ether-options lacp system-priority 100
set groups __contrail__ interfaces ae127 aggregated-ether-options lacp system-id 44:55:66:77:88:11
set groups __contrail__ interfaces ae127 aggregated-ether-options lacp admin-key 1
set groups __contrail__ interfaces ae127 unit 100 vlan-id 100
set groups __contrail__ interfaces xe-0/0/46 gigether-options 802.3ad ae127
set groups __contrail__ interfaces ae126 flexible-vlan-tagging
set groups __contrail__ interfaces ae126 encapsulation extended-vlan-bridge
set groups __contrail__ interfaces ae126 esi 00:11:22:33:44:55:66:77:88:99
set groups __contrail__ interfaces ae126 esi all-active
set groups __contrail__ interfaces ae126 aggregated-ether-options lacp active
set groups __contrail__ interfaces ae126 aggregated-ether-options lacp system-priority 100
set groups __contrail__ interfaces ae126 aggregated-ether-options lacp system-id 44:55:66:77:88:99
set groups __contrail__ interfaces ae126 aggregated-ether-options lacp admin-key 1
set groups __contrail__ interfaces ae126 unit 100 vlan-id 100
set groups __contrail__ interfaces xe-0/0/1 gigether-options 802.3ad ae126

Closes-Bug: #1732327

Change-Id: I7a1e6dab96af21f24c840165747a84b822a93061
(cherry picked from commit 563def507677af33f54292d299d8354b5ec79613)

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

Review in progress for https://review.opencontrail.org/38531
Submitter: Suresh Balineni (<email address hidden>)

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

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

commit f517d410366c971a5ecde98438ebfc70bc5ae17d
Author: sbalineni <email address hidden>
Date: Wed Nov 15 12:56:50 2017 -0800

[DM]: QFX5k - ae allocation: reserve/free right index values

reserve/free right indexes, in the case of reverse allocation of bitarray

generated config (ae127, ae126):

set groups __contrail__ interfaces ae127 flexible-vlan-tagging
set groups __contrail__ interfaces ae127 encapsulation extended-vlan-bridge
set groups __contrail__ interfaces ae127 esi 00:11:22:33:44:55:66:77:88:11
set groups __contrail__ interfaces ae127 esi all-active
set groups __contrail__ interfaces ae127 aggregated-ether-options lacp active
set groups __contrail__ interfaces ae127 aggregated-ether-options lacp system-priority 100
set groups __contrail__ interfaces ae127 aggregated-ether-options lacp system-id 44:55:66:77:88:11
set groups __contrail__ interfaces ae127 aggregated-ether-options lacp admin-key 1
set groups __contrail__ interfaces ae127 unit 100 vlan-id 100
set groups __contrail__ interfaces xe-0/0/46 gigether-options 802.3ad ae127
set groups __contrail__ interfaces ae126 flexible-vlan-tagging
set groups __contrail__ interfaces ae126 encapsulation extended-vlan-bridge
set groups __contrail__ interfaces ae126 esi 00:11:22:33:44:55:66:77:88:99
set groups __contrail__ interfaces ae126 esi all-active
set groups __contrail__ interfaces ae126 aggregated-ether-options lacp active
set groups __contrail__ interfaces ae126 aggregated-ether-options lacp system-priority 100
set groups __contrail__ interfaces ae126 aggregated-ether-options lacp system-id 44:55:66:77:88:99
set groups __contrail__ interfaces ae126 aggregated-ether-options lacp admin-key 1
set groups __contrail__ interfaces ae126 unit 100 vlan-id 100
set groups __contrail__ interfaces xe-0/0/1 gigether-options 802.3ad ae126

Unit Tests Added

Closes-Bug: #1732327

Change-Id: I7a1e6dab96af21f24c840165747a84b822a93061

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.