Skip to content

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

Merged
StormLiangMS merged 3 commits intosonic-net:masterfrom
ccroy-arista:no-vxlan-topos
Sep 16, 2025
Merged

[TH5] Skip VxLAN Tests for Non-Applicable Topologies#20552
StormLiangMS merged 3 commits intosonic-net:masterfrom
ccroy-arista:no-vxlan-topos

Conversation

@ccroy-arista
Copy link
Contributor

@ccroy-arista ccroy-arista commented Sep 7, 2025 •
edited
Loading

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
  • 202412
  • 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.

@ccroy-arista
[TH5] Skip VxLAN Tests for Non-Applicable Topologies
044c36d 
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
@mssonicbld
Copy link
Collaborator

mssonicbld commented Sep 7, 2025

/azp run

@azure-pipelines
Copy link

azure-pipelines bot commented Sep 7, 2025

Azure Pipelines successfully started running 1 pipeline(s).

sdszhang
sdszhang approved these changes Sep 9, 2025
View reviewed changes
Copy link
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

@mssonicbld
Copy link
Collaborator

mssonicbld commented Sep 9, 2025

/azp run

@azure-pipelines
Copy link

azure-pipelines bot commented Sep 9, 2025

Azure Pipelines successfully started running 1 pipeline(s).

This was referenced Sep 9, 2025
Closed
Merged
wangxin
wangxin previously approved these changes Sep 11, 2025
View reviewed changes
@StormLiangMS
Copy link
Collaborator

StormLiangMS commented Sep 11, 2025

hi @ccroy-arista could you check the static check failure?

@sdszhang
Copy link
Contributor

sdszhang commented Sep 12, 2025

Hi @ccroy-arista, the pipeline issue is resolved now. can you resolve the conflict?

@mssonicbld
Copy link
Collaborator

mssonicbld commented Sep 12, 2025

/azp run

@azure-pipelines
Copy link

azure-pipelines bot commented Sep 12, 2025

Azure Pipelines successfully started running 1 pipeline(s).

@ccroy-arista
Copy link
Contributor Author

ccroy-arista commented Sep 12, 2025

Hi @ccroy-arista, the pipeline issue is resolved now. can you resolve the conflict?

Conflict resolved.

@sdszhang sdszhang mentioned this pull request Sep 15, 2025
Merged
11 tasks
StormLiangMS
StormLiangMS approved these changes Sep 16, 2025
View reviewed changes
Copy link
Collaborator

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

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

LGTM

@StormLiangMS StormLiangMS merged commit b1ea0a9 into sonic-net:master Sep 16, 2025
21 checks passed
@ccroy-arista ccroy-arista deleted the no-vxlan-topos branch September 16, 2025 05:42
@Pterosaur
Copy link
Contributor

Pterosaur commented Sep 17, 2025

Hi @ccroy-arista , could you please help to create a PR for 202412?
https://github.com/Azure/sonic-mgmt.msft/tree/202412

@mssonicbld
Copy link
Collaborator

mssonicbld commented Sep 17, 2025

@ccroy-arista PR conflicts with 202505 branch

@ccroy-arista ccroy-arista mentioned this pull request Sep 17, 2025
Merged
10 tasks
@ccroy-arista
Copy link
Contributor Author

ccroy-arista commented Sep 17, 2025

Cherry-pick for 202505: #20708

@ccroy-arista
Copy link
Contributor Author

ccroy-arista commented Sep 17, 2025

Hi @ccroy-arista , could you please help to create a PR for 202412? https://github.com/Azure/sonic-mgmt.msft/tree/202412

@Pterosaur
Once Azure/sonic-mgmt.msft#716 is merged I will post the cherry-pick to 202412.

@Pterosaur
Copy link
Contributor

Pterosaur commented Sep 17, 2025

Hi @ccroy-arista , could you please help to create a PR for 202412? https://github.com/Azure/sonic-mgmt.msft/tree/202412

@Pterosaur Once Azure/sonic-mgmt.msft#716 is merged I will post the cherry-pick to 202412.

Hi @ccroy-arista , thanks for your reply. I just merged it.

@ccroy-arista ccroy-arista mentioned this pull request Sep 17, 2025
Merged
10 tasks
@ccroy-arista
Copy link
Contributor Author

ccroy-arista commented Sep 17, 2025

@Pterosaur here is the cherry-pick for 202412: Azure/sonic-mgmt.msft#718

Pterosaur pushed a commit to Azure/sonic-mgmt.msft that referenced this pull request Sep 17, 2025
@ccroy-arista
[TH5] Skip VxLAN Tests for Non-Applicable Topologies (#718)
659a17b 
Note: this is a manual cherry-pick to msft-202412 of
sonic-net/sonic-mgmt#20552
Note: depends on sonic-net/sonic-mgmt#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
- [x] Test case improvement

### Back port request
- [ ] 202205
- [ ] 202305
- [ ] 202311
- [ ] 202405
- [x] 202412

### 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.
dayouliu1 pushed a commit to dayouliu1/sonic-mgmt that referenced this pull request Sep 18, 2025
@ccroy-arista @dayouliu1
[TH5] Skip VxLAN Tests for Non-Applicable Topologies (sonic-net#20552)
4703e32 
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
yejianquan pushed a commit that referenced this pull request Sep 18, 2025
@ccroy-arista
[TH5] Skip VxLAN Tests for Non-Applicable Topologies (#20708)
0f5942b 
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.

signed-off-by: [email protected]
vidyac86 pushed a commit to vidyac86/sonic-mgmt that referenced this pull request Oct 23, 2025
@ccroy-arista @vidyac86
[TH5] Skip VxLAN Tests for Non-Applicable Topologies (sonic-net#20552)
1db22eb 
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
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
@ccroy-arista
[TH5] Skip VxLAN Tests for Non-Applicable Topologies (sonic-net#20552)
eadb025 
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

Signed-off-by: opcoder0 <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
@ccroy-arista @gshemesh2
[TH5] Skip VxLAN Tests for Non-Applicable Topologies (sonic-net#20552)
898483d 
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

Signed-off-by: Guy Shemesh <[email protected]>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
@ccroy-arista @AharonMalkin
[TH5] Skip VxLAN Tests for Non-Applicable Topologies (sonic-net#20552)
c5afbd9 
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

Signed-off-by: Aharon Malkin <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
@ccroy-arista @gshemesh2
[TH5] Skip VxLAN Tests for Non-Applicable Topologies (sonic-net#20552)
e126773 
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

Signed-off-by: Guy Shemesh <[email protected]>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
@ccroy-arista @venu-nexthop
[TH5] Skip VxLAN Tests for Non-Applicable Topologies (sonic-net#20552)
448f816 
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
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
@ccroy-arista @gshemesh2
[TH5] Skip VxLAN Tests for Non-Applicable Topologies (sonic-net#20552)
29e3da9 
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

Signed-off-by: Guy Shemesh <[email protected]>
lakshmi-nexthop pushed a commit to lakshmi-nexthop/sonic-mgmt that referenced this pull request Jan 28, 2026
@ccroy-arista @lakshmi-nexthop
[TH5] Skip VxLAN Tests for Non-Applicable Topologies (sonic-net#20552)
17ddd6a 
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

Signed-off-by: Lakshmi Yarramaneni <[email protected]>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
@ccroy-arista @ytzur1
[TH5] Skip VxLAN Tests for Non-Applicable Topologies (sonic-net#20552)
ba631bf 
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

Signed-off-by: Yael Tzur <[email protected]>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
@pavannaregundi
[Marvell] Correct the platform asic name (sonic-net#20552)
3b57d7f 
Signed-off-by: Pavan Naregundi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@StormLiangMS StormLiangMS StormLiangMS approved these changes

@sdszhang sdszhang sdszhang approved these changes

@wangxin wangxin wangxin left review comments

Assignees

No one assigned

Labels

Approved for msft-202412 Branch Approved for 202505 branch Included in msft-202412 Branch Included in 202505 branch Request for msft-202412 Branch Request for 202505 branch

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@ccroy-arista @mssonicbld @StormLiangMS @sdszhang @Pterosaur @wangxin @yejianquan