Adjust test_bgpmon.py to handle running over ipv6 only topologies#21377
Merged
bingwang-ms merged 11 commits intosonic-net:masterfrom Jan 4, 2026
Merged
Adjust test_bgpmon.py to handle running over ipv6 only topologies#21377bingwang-ms merged 11 commits intosonic-net:masterfrom
bingwang-ms merged 11 commits intosonic-net:masterfrom
Conversation
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Contributor
Author
|
/azpw run |
Collaborator
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Commenter does not have sufficient privileges for PR 21377 in repo sonic-net/sonic-mgmt |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Fix typos in comments and method calls related to BGP monitor setup.
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
AharonMalkin
added
Included in 202511 branch
Request for 202511 branch
vrajeshe
pushed a commit
to Akshath-17/sonic-mgmt
that referenced
this pull request
Jan 4, 2026
…nic-net#21377) * Adjust test_bgpmon.py to handle running over ipv6 only topologies Signed-off-by: Venkata Gouri Rajesh Etla <[email protected]>
Collaborator
|
Pick to 202412: Azure/sonic-mgmt.msft#888 |
venu-nexthop
pushed a commit
to venu-nexthop/sonic-mgmt
that referenced
this pull request
Jan 13, 2026
…nic-net#21377) * Adjust test_bgpmon.py to handle running over ipv6 only topologies
yifan-nexthop
pushed a commit
to nexthop-ai/sonic-mgmt
that referenced
this pull request
Jan 14, 2026
…nic-net#21377) * Adjust test_bgpmon.py to handle running over ipv6 only topologies Signed-off-by: YiFan Wang <[email protected]>
vmittal-msft
added
Approved for 202511 branch
and removed
Approved for 202511 branch
labels
mssonicbld
pushed a commit
to mssonicbld/sonic-mgmt
that referenced
this pull request
Jan 20, 2026
…nic-net#21377) * Adjust test_bgpmon.py to handle running over ipv6 only topologies
Collaborator
|
Cherry-pick PR to 202511: #22024 |
11 tasks
PriyanshTratiya
pushed a commit
to PriyanshTratiya/sonic-mgmt
that referenced
this pull request
Jan 21, 2026
…nic-net#21377) * Adjust test_bgpmon.py to handle running over ipv6 only topologies Signed-off-by: Priyansh Tratiya <[email protected]>
r12f
pushed a commit
to Azure/sonic-mgmt.msft
that referenced
this pull request
Jan 23, 2026
…pv6 only topologies #21377 (#888) Manual cherry-pick of PR[ #21377](sonic-net/sonic-mgmt#21377) Adjust test_bgpmon.py to handle running over ipv6 only topologies
r12f
added
Approved for msft-202412 Branch
Included in msft-202412 Branch
and removed
Cherry Pick Conflict_msft-202412
labels
ytzur1
pushed a commit
to ytzur1/sonic-mgmt
that referenced
this pull request
Feb 2, 2026
…nic-net#21377) * Adjust test_bgpmon.py to handle running over ipv6 only topologies Signed-off-by: Yael Tzur <[email protected]>
vmittal-msft
pushed a commit
that referenced
this pull request
Feb 2, 2026
…1377) (#22024) * Adjust test_bgpmon.py to handle running over ipv6 only topologies Co-authored-by: gshemesh2 <[email protected]>
abhishek-nexthop
pushed a commit
to nexthop-ai/sonic-mgmt
that referenced
this pull request
Feb 6, 2026
…nic-net#21377) * Adjust test_bgpmon.py to handle running over ipv6 only topologies
lakshmi-nexthop
pushed a commit
to lakshmi-nexthop/sonic-mgmt
that referenced
this pull request
Feb 11, 2026
…nic-net#21377) (sonic-net#22024) * Adjust test_bgpmon.py to handle running over ipv6 only topologies Co-authored-by: gshemesh2 <[email protected]> Signed-off-by: Lakshmi Yarramaneni <[email protected]>
nnelluri-cisco
pushed a commit
to nnelluri-cisco/sonic-mgmt
that referenced
this pull request
Feb 12, 2026
…nic-net#21377) * Adjust test_bgpmon.py to handle running over ipv6 only topologies Signed-off-by: nnelluri-cisco <[email protected]>
rraghav-cisco
pushed a commit
to rraghav-cisco/sonic-mgmt
that referenced
this pull request
Feb 13, 2026
…nic-net#21377) * Adjust test_bgpmon.py to handle running over ipv6 only topologies Signed-off-by: Raghavendran Ramanathan <[email protected]>
anilal-amd
pushed a commit
to anilal-amd/anilal-forked-sonic-mgmt
that referenced
this pull request
Feb 19, 2026
…nic-net#21377) * Adjust test_bgpmon.py to handle running over ipv6 only topologies Signed-off-by: Zhuohui Tan <[email protected]>
abhishek-nexthop
pushed a commit
to nexthop-ai/sonic-mgmt
that referenced
this pull request
Mar 17, 2026
…nic-net#21377) * Adjust test_bgpmon.py to handle running over ipv6 only topologies Signed-off-by: Abhishek <[email protected]>
Pterosaur
pushed a commit
to Pterosaur/sonic-mgmt
that referenced
this pull request
Mar 26, 2026
``` * d849917 (HEAD -> user/r12f/merge-202412, origin/user/r12f/merge-202412, internal-202412) r12f 260209:0619 - Merge remote-tracking branch 'azure/202412' into internal-202412 |\ | * c4c429c (azure/202412) mssonicbld 260209:0115 - [action] [PR:21843] Fix test_bgp_suppress_fib.py flakiness on scale topos (sonic-net#1011) | * 46503f1 Mark Xiao 260205:1058 - [202412] Fix test_acl.py [ipv6-ingress-uplink->downlink-*] cases for v6 topo (sonic-net#942) | * b6c58d4 mssonicbld 260130:1215 - [action] [PR:21772] Fix test_bgp_suppress_fib.py for v6 topos (sonic-net#985) | * e762451 mssonicbld 260130:1016 - [action] [PR:21762] Fix test_disk_exhaustion.py for v6 topos (sonic-net#983) | * 02cadbb mssonicbld 260130:1016 - [action] [PR:21771] Fix test_bgp_bounce.py for v6 topos (sonic-net#984) | * 7eab3e7 Priyansh 260129:1437 - [202412] Fix/nonlinear dataplane downtime (sonic-net#982) | * de89a46 mssonicbld 260130:0616 - [action] [PR:21143] configlet/test_add_rack.py Add comparison ignore for extra entries added by generic patcher (sonic-net#977) | * c731442 mssonicbld 260130:0616 - [action] [PR:21647] Remove test_route_flow_counter.py xfail for v6 topos (sonic-net#979) | * 0ff3ecc mssonicbld 260130:0116 - [action] [PR:21717] [BGP][test_bgp_session.py::test_bgp_session_interface_down] - Increase BGP Session State Timeout Window when restarting SWSS Container (sonic-net#978) | * e0a408c Mark Xiao 260129:0914 - [202412] Fix fib/test_fib.py test_vxlan_hash [ipv6-*] (sonic-net#971) | * 51b0525 mssonicbld 260124:0615 - [action] [PR:21523] Feature/route programming data (sonic-net#974) | * 2bb2a53 mssonicbld 260124:0615 - [action] [PR:21939] Fix/nonlinear high nexthop dataplane downtime (sonic-net#975) | * 1fe73bc Mark Xiao 260123:1323 - [202412] Fix setup_interfaces fixture and related bgp tests for v6 topo (sonic-net#941) | * fda4ac2 Chuan Wu 260124:0520 - Enhance PTF function at scale testbed (sonic-net#913) | * e8edcae gshemesh2 260123:2317 - Manual cherry-pick PR: Adjust test_bgpmon.py to handle running over ipv6 only topologies sonic-net#21377 (sonic-net#888) | * 24b3dc7 gshemesh2 260123:2313 - Manual cherry-pick of PR: Adjusting test_stress_arp is it can handle cases of ipv6-only topologies . sonic-net#20932 (sonic-net#923) ``` ---- #### AI description (iteration 1) #### PR Classification This pull request is a feature enhancement that adds and refines IPv6 compatibility across multiple BGP, ACL, and related test modules. #### PR Summary The changes extend the testing framework and helper utilities to support IPv6-only and dual-stack environments by conditionally switching commands, packet generation, and route verifications. - `tests/bgp/test_bgpmon.py` and `tests/bgp/test_bgp_update_timer.py`: Updated function signatures and logic to construct, verify, and clean up IPv6 routes and packets (e.g., using /64 and /128 prefixes). - `tests/bgp/test_bgp_suppress_fib.py` and `tests/bgp/test_bgp_peer_shutdown.py`: Modified commands and packet filters to support IPv6 addressing and proper neighbor session handling. - `tests/common/...` files and generators: Integr...
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of PR
Summary:
Adjust test_bgpmon.py to handle running over ipv6 only topologies
Type of change
Back port request
Approach
What is the motivation for this PR?
Adjust test_bgpmon.py to handle running over ipv6 only topologies.
How did you do it?
Adjust test_bgpmon.py to handle running over ipv6 only topologies, by adjusting the commands for both ipv4 and ipv6 and create new function for ignore extra bytes for ipv6 packets.
How did you verify/test it?
tested on Nvidia platforms, with t0 and t1 ipv6 only and non-ipv6 only topologies.
Any platform specific information?
N/A
Supported testbed topology if it's a new test case?
any, tested on t1 and t0 topologies
Documentation