Skip to content

Get bgp neighbor information only if bgp service is in critical process #7046

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
SuvarnaMeenakshi wants to merge 2 commits into from
Closed

Get bgp neighbor information only if bgp service is in critical process #7046

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
236081e
Get bgp neighbor information only if bgp service is enabled
SuvarnaMeenakshi Dec 16, 2022
978f9e7
Fix space
SuvarnaMeenakshi Dec 16, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions tests/common/devices/multi_asic.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -482,6 +482,8 @@ def get_bgp_neighbors(self):
"""
bgp_neigh = {}
for asic in self.asics:
if asic.get_docker_name("bgp") not in self.critical_services:
continue
bgp_info = asic.bgp_facts()
bgp_neigh.update(bgp_info["ansible_facts"]["bgp_neighbors"])

Expand All @@ -499,6 +501,8 @@ def get_bgp_neighbors_per_asic(self, state="established"):
bgp_neigh = {}
for asic in self.asics:
bgp_neigh[asic.namespace] = {}
if asic.get_docker_name("bgp") not in self.critical_services:
continue
bgp_info = asic.bgp_facts()["ansible_facts"]["bgp_neighbors"]
for k, v in bgp_info.items():
if v["state"] != state:
Expand All @@ -518,6 +522,8 @@ def check_bgp_session_state(self, neigh_ips, state="established"):
neigh_ok = []

for asic in self.asics:
if asic.get_docker_name("bgp") not in self.critical_services:
continue
bgp_facts = asic.bgp_facts()['ansible_facts']
for k, v in bgp_facts['bgp_neighbors'].items():
if v['state'] == state:
Expand All @@ -538,6 +544,8 @@ def check_bgp_session_state_all_asics(self, bgp_neighbors, state="established"):
@param state: target state
"""
for asic in self.asics:
if asic.get_docker_name("bgp") not in self.critical_services:
continue
if asic.namespace in bgp_neighbors:
neigh_ips = [k.lower() for k, v in bgp_neighbors[asic.namespace].items() if v["state"] == state]
if not asic.check_bgp_session_state(neigh_ips, state):
Expand Down