Add Multicast EdgeDiscovery and EdgeForwarding path attributes to show route introspect

Bug #1696856 reported by Ananth Suryanarayana
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.1
Fix Committed
Medium
Nikhil Bansal
R3.2
Fix Committed
Medium
Nikhil Bansal
R4.0
Fix Committed
Medium
Nikhil Bansal
Trunk
Fix Committed
Medium
Nikhil Bansal

Bug Description

If Multicast EdgeDiscovery and/or EdgeForwarding path attributes are present in a path, they must be shown in a user friendly way in both introspect and in contrail UI.

Currently, introspect page does not show this. UI also does not show those attributes.

e.g.

1-0:0-172.17.90.3,255.255.255.255,0.0.0.0BGP172.17.90.30.0.0.00-default-domain:admin:VN10
 {
protocol: BGP
last_modified: 2017-Jun-08 20:33:44.841023
local_preference: 100
med: 0
local_as: 64513
peer_as: 64513
peer_router_id: 172.17.90.3
source: 172.17.90.3
as_path:
next_hop: 0.0.0.0
label: 0
replicated: true
primary_table: bgp.ermvpn.0
secondary_tables: {
list:
}
communities: {
list: { ... }
}
origin_vn: default-domain:admin:VN10
flags: {
list: { ... }
}
tunnel_encap: {
list:
}
sequence_no:
origin_vn_path: {
list:
}
pmsi_tunnel: {
ShowPmsiTunnel: { ... }
}
load_balance: {
ShowLoadBalance: { ... }
}
cluster_list: {
list:
}
}

Nischal Sheth (nsheth)
Changed in juniperopenstack:
importance: Undecided → Medium
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/34193
Submitter: Nikhil Bansal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/34336
Submitter: Nikhil Bansal (<email address hidden>)

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

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

commit 23c420fb8649605e419570f49aa6ac4f4dd5c576
Author: Nikhil B <email address hidden>
Date: Tue Aug 1 10:48:17 2017 +0530

Adding EdgeForwarding and EdgeDiscovery attributes

If Multicast EdgeDiscovery and/or EdgeForwarding path attributes are
present in a path, they must be shown in a user friendly way in
introspect pages. Those attributes are being added in corresponding
sandesh response.
Taking care of comments
Taking care of Nischal's comments

Change-Id: Ia93ebc5de7316c025db19c86ded5bdb579deea9e
Closes-Bug: #1696856

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

Review in progress for https://review.opencontrail.org/34346
Submitter: Nikhil Bansal (<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/34347
Submitter: Nikhil Bansal (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/34336
Submitter: Nikhil Bansal (<email address hidden>)

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

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

commit 13b19678069326ad3ec6ca2b97eea35b3e19116f
Author: Nikhil B <email address hidden>
Date: Tue Aug 1 10:48:17 2017 +0530

Adding EdgeForwarding and EdgeDiscovery attributes

If Multicast EdgeDiscovery and/or EdgeForwarding path attributes are
present in a path, they must be shown in a user friendly way in
introspect pages. Those attributes are being added in corresponding
sandesh response.
Taking care of comments
Taking care of Nischal's comments

Change-Id: Ia93ebc5de7316c025db19c86ded5bdb579deea9e
Closes-Bug: #1696856
(cherry picked from commit 23c420fb8649605e419570f49aa6ac4f4dd5c576)

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

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

commit 02a1bb31e76bf67c1be1acf5e65d8ec890a977ae
Author: Nikhil B <email address hidden>
Date: Tue Aug 1 10:48:17 2017 +0530

Adding EdgeForwarding and EdgeDiscovery attributes

If Multicast EdgeDiscovery and/or EdgeForwarding path attributes are
present in a path, they must be shown in a user friendly way in
introspect pages. Those attributes are being added in corresponding
sandesh response.
Taking care of comments
Taking care of Nischal's comments

Change-Id: Ia93ebc5de7316c025db19c86ded5bdb579deea9e
Closes-Bug: #1696856
(cherry picked from commit 23c420fb8649605e419570f49aa6ac4f4dd5c576)

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

Review in progress for https://review.opencontrail.org/34376
Submitter: Nikhil Bansal (<email address hidden>)

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

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

commit 1f3971a4e4db021e918df9440431318d635fcace
Author: Nikhil B <email address hidden>
Date: Sun Aug 6 08:57:01 2017 +0530

Adding SourceAS and RtImport extended communities

These 2 new communities are needed for MVPN support.
patch3: Taking care of Nischal's comments
patch4: Adding testcases for Type2 in SourceAsTest

Change-Id: Ia30872df0cf1ef274cb56647219831bb8c7ef73b
Partial-Bug: #1696856

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

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

commit e5e7dfdf18bd89099837a61d8c0aa1a8e424e8ca
Author: Nikhil B <email address hidden>
Date: Tue Aug 1 10:48:17 2017 +0530

Adding EdgeForwarding and EdgeDiscovery attributes

If Multicast EdgeDiscovery and/or EdgeForwarding path attributes are
present in a path, they must be shown in a user friendly way in
introspect pages. Those attributes are being added in corresponding
sandesh response.
Taking care of comments
Taking care of Nischal's comments

Closes-Bug: #1696856
(cherry picked from commit 23c420fb8649605e419570f49aa6ac4f4dd5c576)

Conflicts:
 src/bgp/bgp_peer.sandesh
 src/bgp/bgp_route.cc

Change-Id: I2ce1f94fda4e7ee3f99d59625a5e0d7d8c465882

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.