Skip to content

Comments

[action] [PR:21377] Adjust test_bgpmon.py to handle running over ipv6 only topologies#22024

Merged
vmittal-msft merged 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/21377
Feb 2, 2026
Merged

[action] [PR:21377] Adjust test_bgpmon.py to handle running over ipv6 only topologies#22024
vmittal-msft merged 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/21377

Conversation

@mssonicbld
Copy link
Collaborator

Description of PR

Summary:
Adjust test_bgpmon.py to handle running over ipv6 only topologies

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
  • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

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

…nic-net#21377)

* Adjust test_bgpmon.py to handle running over ipv6 only topologies
@mssonicbld
Copy link
Collaborator Author

Original PR: #21377

@vmittal-msft vmittal-msft merged commit 1cd419b into sonic-net:202511 Feb 2, 2026
5 checks passed
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 <gshemesh@nvidia.com>
Signed-off-by: Lakshmi Yarramaneni <lakshmi@nexthop.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants