Skip to content

Commit c04239a

Browse files
committed
[PR#17130] - BGP Queue - Changes for VOQ single-dut multi-asic
<!-- Please make sure you've read and understood our contributing guidelines; https://github.com/sonic-net/SONiC/blob/gh-pages/CONTRIBUTING.md Please provide following information to help code review process a bit easier: --> ### Description of PR <!-- - Please include a summary of the change and which issue is fixed. - Please also include relevant motivation and context. Where should reviewer start? background context? - List any dependencies that are required for this change. --> Summary: Fixes # (issue) - Split the original PR sonic-net/sonic-mgmt#17130 into individual PRs for each test. - This is an enhancement to support VOQ Single DUT Multi-ASIC setup for T2 topo. ### Type of change <!-- - Fill x for your type of change. - e.g. - [x] Bug fix --> - [ ] Bug fix - [ ] Testbed and Framework(new/improvement) - [ ] New Test case - [ ] Skipped for non-supported platforms - [x] Test case improvement ### Back port request - [ ] 202012 - [ ] 202205 - [ ] 202305 - [ ] 202311 - [ ] 202405 - [x] 202411 ### Approach #### What is the motivation for this PR? - New testbed modification of VOQ Single DUT and multi-asic. #### How did you do it? - Skip internal neighbors for both VOQ single dut multi-asic as well as multi-dut multi-asic. #### How did you verify/test it? #### Any platform specific information? #### Supported testbed topology if it's a new test case? - T2 VOQ Single Dut Multi ASIC ### Documentation <!-- (If it's a new feature, new test case) Did you update documentation/Wiki relevant to your implementation? Link to the wiki page? -->
1 parent d50da0c commit c04239a

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

tests/bgp/test_bgp_queue.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,8 +64,7 @@ def test_bgp_queues(duthosts, enum_frontend_dut_hostname, enum_asic_index, tbinf
6464
# or "VOQ_CHASSIS_PEER_V4" or "VOQ_CHASSIS_PEER_V6" for VOQ_CHASSIS
6565
# If it's external it will be "RH_V4", "RH_V6", "AH_V4", "AH_V6", ...
6666
# Skip internal neighbors for VOQ_CHASSIS until BRCM fixes iBGP traffic in 2024011
67-
if ("INTERNAL" in v["peer group"] or 'VOQ_CHASSIS' in v["peer group"]) and \
68-
duthost.get_facts().get('modular_chassis'):
67+
if ("INTERNAL" in v["peer group"] or 'VOQ_CHASSIS' in v["peer group"]):
6968
# Skip iBGP neighbors since we only want to verify eBGP
7069
continue
7170
assert (k in arp_dict.keys() or k in ndp_dict.keys())

0 commit comments

Comments
 (0)