Skip to content

[202412] Fix setup_interfaces fixture and related bgp tests for v6 topo#941

Merged
r12f merged 1 commit intoAzure:202412from
markx-arista:fix-setup-interfaces-for-bgp-tests-v6-topo
Jan 23, 2026
Merged

[202412] Fix setup_interfaces fixture and related bgp tests for v6 topo#941
r12f merged 1 commit intoAzure:202412from
markx-arista:fix-setup-interfaces-for-bgp-tests-v6-topo

Conversation

@markx-arista
Copy link

@markx-arista markx-arista commented Dec 23, 2025

Description of PR

Summary:
Fix setup_interfaces fixture and related bgp tests for v6 topo:
test_bgp_update_timer.py
test_bgp_peer_shutdown.py
test_traffic_shift.py
test_bgp_allow_list.py

In general:
Skip IPv4 checks for v6 topos
Fix DUT/PTF commands for IPv6
Fix len range error for for IPv6 IPNetwork in generate_ips by using .size
Fix scapy parser for IPv6 BGP packets
For IPv6 router ID of PTF neighbors, use 20.0.0.0 as base and combine it with last 3 bytes of IPv6 addr

Manual Cherry pick of sonic-net/sonic-mgmt#21577

Type of change

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

Back port request

  • 202412
  • 202505

Approach

What is the motivation for this PR?

Fix setup_interfaces fixture and related bgp tests for v6 topo

How did you do it?

See summary

How did you verify/test it?

Run the tests on t0/t1-isolated-v6-* topos

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

Fix the following bgp tests:
test_bgp_update_timer.py
test_bgp_peer_shutdown.py
test_traffic_shift.py
test_bgp_allow_list.py

In general:
Skip IPv4 checks for v6 topos
Fix DUT/PTF commands for IPv6
Fix len range error for for IPv6 IPNetwork in generate_ips by using .size
Fix scapy parser for IPv6 BGP packets
For IPv6 router ID of PTF neighbors, use 20.0.0.0 as base and combine it with last 3 bytes of IPv6 addr

Signed-off-by: markxiao <markxiao@arista.com>
@markx-arista markx-arista force-pushed the fix-setup-interfaces-for-bgp-tests-v6-topo branch from 0712305 to df1b671 Compare December 23, 2025 20:09
@r12f
Copy link
Contributor

r12f commented Jan 9, 2026

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@markx-arista
Copy link
Author

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 941 in repo Azure/sonic-mgmt.msft

@r12f
Copy link
Contributor

r12f commented Jan 23, 2026

/azp run

@r12f r12f merged commit 1fe73bc into Azure:202412 Jan 23, 2026
12 of 14 checks passed
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants