From 24e509b83fc722a53b1eef583bd08d1f5f2d4780 Mon Sep 17 00:00:00 2001 From: Lukas Stockner Date: Mon, 16 May 2022 23:46:21 +0200 Subject: [PATCH] Skip unnumbered BGP neighbors instead of failing --- src/sonic_ax_impl/mibs/vendor/cisco/bgp4.py | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/sonic_ax_impl/mibs/vendor/cisco/bgp4.py b/src/sonic_ax_impl/mibs/vendor/cisco/bgp4.py index 87214707b..46dd68bf3 100644 --- a/src/sonic_ax_impl/mibs/vendor/cisco/bgp4.py +++ b/src/sonic_ax_impl/mibs/vendor/cisco/bgp4.py @@ -39,7 +39,13 @@ def update_data(self): neigh_info = self.db_conn[db_index].get_all(mibs.STATE_DB, neigh_key, blocking=False) if neigh_info: state = neigh_info['state'] - ip = ipaddress.ip_address(neigh_str) + try: + ip = ipaddress.ip_address(neigh_str) + except ValueError: + # In case of unnumbered BGP, the neighbor is an interface. + # That can't be represented here, so just skip the neighbor. + continue + if type(ip) is ipaddress.IPv4Address: oid_head = (1, 4) else: