Skip to content

Commit 75922b0

Browse files
authored
fix: only test eBGP neighbors (sonic-net#14310)
Description of PR Summary: only test eBGP neighbors for test_bgp_queues Approach What is the motivation for this PR? Complete sonic-net#11116 How did you do it? How did you verify/test it? T2 profile Signed-off-by: Austin Pham <austinpham@microsoft.com>
1 parent 7b5156e commit 75922b0

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

tests/bgp/test_bgp_queue.py

+5
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,11 @@ def test_bgp_queues(duthosts, enum_frontend_dut_hostname, enum_asic_index, tbinf
5959
for k, v in list(bgp_facts['bgp_neighbors'].items()):
6060
# Only consider established bgp sessions
6161
if v['state'] == 'established':
62+
# For "peer group" if it's internal it will be "INTERNAL_PEER_V4" or "INTERNAL_PEER_V6"
63+
# If it's external it will be "RH_V4", "RH_V6", "AH_V4", "AH_V6", ...
64+
if "INTERNAL" in v["peer group"] and duthost.get_facts().get('modular_chassis'):
65+
# Skip iBGP neighbors since we only want to verify eBGP
66+
continue
6267
assert (k in arp_dict.keys() or k in ndp_dict.keys())
6368
if k in arp_dict:
6469
ifname = arp_dict[k].split('.', 1)[0]

0 commit comments

Comments
 (0)