openstack/os-ken
Code Issues Proposed changes

bgp: support new handler in case of changing BGP session

Browse Source 
When BGP session goes up/down, BGPSpeaker can detect changing the session.

Signed-off-by: Toshiki Tsuboi <t.tsubo2000@gmail.com>
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
This commit is contained in:
Toshiki Tsuboi 2014-12-06 02:46:33 +09:00 committed by FUJITA Tomonori
parent 567ff4ec17
commit 3c03ba0034
2 changed files with 39 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
doc/source/library_bgp_speaker.rst
View File

@ -38,9 +38,13 @@ instance advertizes some prefixes.
print 'the best path changed:', event.remote_as, event.prefix,\ print 'the best path changed:', event.remote_as, event.prefix,\
event.nexthop, event.is_withdraw event.nexthop, event.is_withdraw
def detect_peer_down(remote_ip, remote_as):
print 'Peer down:', remote_ip, remote_as
if __name__ == "__main__": if __name__ == "__main__":
speaker = BGPSpeaker(as_number=64512, router_id='10.0.0.1', speaker = BGPSpeaker(as_number=64512, router_id='10.0.0.1',
best_path_change_handler=dump_remote_best_path_change) best_path_change_handler=dump_remote_best_path_change,
peer_down_handler=detect_peer_down)
speaker.neighbor_add('192.168.177.32', 64513) speaker.neighbor_add('192.168.177.32', 64513)
# uncomment the below line if the speaker needs to talk with a bmp server. # uncomment the below line if the speaker needs to talk with a bmp server.

36
ryu/services/protocols/bgp/bgpspeaker.py
View File

@ -108,6 +108,8 @@ class BGPSpeaker(object):
refresh_stalepath_time=DEFAULT_REFRESH_STALEPATH_TIME, refresh_stalepath_time=DEFAULT_REFRESH_STALEPATH_TIME,
refresh_max_eor_time=DEFAULT_REFRESH_MAX_EOR_TIME, refresh_max_eor_time=DEFAULT_REFRESH_MAX_EOR_TIME,
best_path_change_handler=None, best_path_change_handler=None,
peer_down_handler=None,
peer_up_handler=None,
ssh_console=False, ssh_console=False,
label_range=DEFAULT_LABEL_RANGE): label_range=DEFAULT_LABEL_RANGE):
"""Create a new BGPSpeaker object with as_number and router_id to """Create a new BGPSpeaker object with as_number and router_id to
@ -137,6 +139,12 @@ class BGPSpeaker(object):
peer down. The handler is supposed to take one argument, the peer down. The handler is supposed to take one argument, the
instance of an EventPrefix class instance. instance of an EventPrefix class instance.
``peer_down_handler``, if specified, is called when BGP peering
session goes down.
``peer_up_handler``, if specified, is called when BGP peering
session goes up.
""" """
super(BGPSpeaker, self).__init__() super(BGPSpeaker, self).__init__()
@ -150,9 +158,23 @@ class BGPSpeaker(object):
self._core_start(settings) self._core_start(settings)
self._init_signal_listeners() self._init_signal_listeners()
self._best_path_change_handler = best_path_change_handler self._best_path_change_handler = best_path_change_handler
self._peer_down_handler = peer_down_handler
self._peer_up_handler = peer_up_handler
if ssh_console: if ssh_console:
hub.spawn(ssh.SSH_CLI_CONTROLLER.start) hub.spawn(ssh.SSH_CLI_CONTROLLER.start)
def _notify_peer_down(self, peer):
remote_ip = peer.protocol.recv_open_msg.bgp_identifier
remote_as = peer.protocol.recv_open_msg.my_as
if self._peer_down_handler:
self._peer_down_handler(remote_ip, remote_as)
def _notify_peer_up(self, peer):
remote_ip = peer.protocol.recv_open_msg.bgp_identifier
remote_as = peer.protocol.recv_open_msg.my_as
if self._peer_up_handler:
self._peer_up_handler(remote_ip, remote_as)
def _notify_best_path_changed(self, path, is_withdraw): def _notify_best_path_changed(self, path, is_withdraw):
if path.source: if path.source:
nexthop = path.nexthop nexthop = path.nexthop
@ -182,8 +204,18 @@ class BGPSpeaker(object):
CORE_MANAGER.get_core_service()._signal_bus.register_listener( CORE_MANAGER.get_core_service()._signal_bus.register_listener(
BgpSignalBus.BGP_BEST_PATH_CHANGED, BgpSignalBus.BGP_BEST_PATH_CHANGED,
lambda _, info: lambda _, info:
self._notify_best_path_changed(info['path'], self._notify_best_path_changed(info['path'],
info['is_withdraw']) info['is_withdraw'])
)
CORE_MANAGER.get_core_service()._signal_bus.register_listener(
BgpSignalBus.BGP_ADJ_DOWN,
lambda _, info:
self._notify_peer_down(info['peer'])
)
CORE_MANAGER.get_core_service()._signal_bus.register_listener(
BgpSignalBus.BGP_ADJ_UP,
lambda _, info:
self._notify_peer_up(info['peer'])
) )
def _core_start(self, settings): def _core_start(self, settings):