Skip to content

[TH5] Skip VxLAN Tests for Non-Applicable Topologies#20708

Merged
yejianquan merged 1 commit intosonic-net:202505from
ccroy-arista:no-vxlan-topos
Sep 18, 2025
Merged

[TH5] Skip VxLAN Tests for Non-Applicable Topologies#20708
yejianquan merged 1 commit intosonic-net:202505from
ccroy-arista:no-vxlan-topos

Conversation

@ccroy-arista
Copy link
Copy Markdown
Contributor

Note: this is a manual cherry-pick to 202505 of #20552
Note: depends on #20382 to pass static analysis.

Description of PR

Skip the following tests:
decap/test_decap.py::test_decap[ttl=pipe, dscp=uniform, vxlan=set_unset]
vxlan/test_vxlan_decap.py

for the following topologes:

  • t0-isolated-d32u32s2
  • t0-isolated-d256u256s2
  • t0-isolated-v6-d32u32s2
  • t0-isolated-v6-d256u256s2
  • t1-isolated-d56u2
  • t1-isolated-d56u1-lag
  • t1-isolated-d448u15-lag
  • t1-isolated-v6-d56u2
  • t1-isolated-v6-d56u1-lag
  • t1-isolated-v6-d448u15-lag
  • t1-isolated-d32
  • t1-isolated-d128
  • t1-isolated-v6-d128
  • t0-isolated-d96u32s2
  • t0-isolated-v6-d96u32s2

Summary:
Fixes # (issue)

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
  • 202505

Approach

What is the motivation for this PR?

Skip tests that are not required for the following topologies: t0-isolated-d32u32s2, t0-isolated-d256u256s2, t0-isolated-v6-d32u32s2, t0-isolated-v6-d256u256s2, t1-isolated-d56u2, t1-isolated-d56u1-lag, t1-isolated-d448u15-lag, t1-isolated-v6-d56u2, t1-isolated-v6-d56u1-lag, t1-isolated-v6-d448u15-lag, t1-isolated-d32, t1-isolated-d128, t1-isolated-v6-d128, t0-isolated-d96u32s2, t0-isolated-v6-d96u32s2.

How did you do it?

Added a skip conditional mark to the following tests:
decap/test_decap.py::test_decap[ttl=pipe, dscp=uniform, vxlan=set_unset]
vxlan/test_vxlan_decap.py

How did you verify/test it?

Tried to run the test and confirmed it is skipped.

Skip the following tests:
decap/test_decap.py::test_decap[ttl=pipe, dscp=pipe, vxlan=disable]
vxlan/test_vxlan_decap.py

for the following topologes:
- t0-isolated-d32u32s2
- t0-isolated-d256u256s2
- t0-isolated-v6-d32u32s2
- t0-isolated-v6-d256u256s2
- t1-isolated-d56u2
- t1-isolated-d56u1-lag
- t1-isolated-d448u15-lag
- t1-isolated-v6-d56u2
- t1-isolated-v6-d56u1-lag
- t1-isolated-v6-d448u15-lag
- t1-isolated-d32
- t1-isolated-d128
- t1-isolated-v6-d128
- t0-isolated-d96u32s2
- t0-isolated-v6-d96u32s2
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Contributor

@sdszhang sdszhang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@yejianquan yejianquan merged commit 0f5942b into sonic-net:202505 Sep 18, 2025
17 checks passed
@ccroy-arista ccroy-arista deleted the no-vxlan-topos branch September 18, 2025 06:42
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.

4 participants