Allow bgp router and bgp peers to be marked admin down

Bug #1514670 reported by Nischal Sheth
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
Fix Committed
Wishlist
Nischal Sheth

Bug Description

Allow bgp router and bgp peers to be marked admin down.

Among other things, this allows an administrator to indicate
intent and can allow generation of alarms, warnings etc.
based on peers being down.

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

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

commit 2e42ea81ad623c7a7978966587acb8dab4287565
Author: Nischal Sheth <email address hidden>
Date: Mon Nov 9 18:01:01 2015 -0800

Add admin_down configuration for bgp router and neighbor

Highlights:

- Add admin-down to BgpRouterParams and BgpSessionAttributes in schema
- BgpPeer is admin down if it's configured to be admin down
- BgpPeer is admin down if local or remote bgp-router is admin down
- Add admin down state to BgpServer
- Reuse existing admin down functionality in BgpPeer and StateMachine
- Include admin down in introspect, tracing, logging and UVEs
- Don't publish to discovery is the bgp-router is admin down
- Bring down all xmpp connections if bgp-router is admin down
- Don't accept xmpp connections if bgp-router is admin down
- Add tests to exercise the new code

Change-Id: I2eae401902f1af6132ddddd28605482eef29de21
Closes-Bug: 1514670

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

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

commit 7934ac4999f716d046a3d23e3dc093a3cbc77868
Author: Nischal Sheth <email address hidden>
Date: Thu Nov 12 10:38:42 2015 -0800

Avoid redundant clear of peer when admin down is changed/set

- Log a message when session is cleared dur to admin down
- Make sure that BgpPeerInfo UVE is updated on admin down change

Change-Id: I74303ce9d078a9b7f7f552b631a0c4eddce0799a
Closes-Bug: 1514670

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.