Skip to content

tests_mark_conditions: xfail test_decap and test_vnet_decap on SN5640#23302

Merged
yxieca merged 1 commit intosonic-net:masterfrom
Pterosaur:zegan/xfail_sn5640_decap_tests
Mar 26, 2026
Merged

tests_mark_conditions: xfail test_decap and test_vnet_decap on SN5640#23302
yxieca merged 1 commit intosonic-net:masterfrom
Pterosaur:zegan/xfail_sn5640_decap_tests

Conversation

@Pterosaur
Copy link
Contributor

@Pterosaur Pterosaur commented Mar 25, 2026

Description of PR

Add xfail mark for decap/test_decap.py::test_decap and vxlan/test_vnet_decap.py::test_vnet_decap on x86_64-nvidia_sn5640-r0 platform because decap is not supported on this platform.

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

Approach

What is the motivation for this PR?

test_decap and test_vnet_decap consistently fail on x86_64-nvidia_sn5640-r0 because decap is not supported on this platform.

How did you do it?

Added xfail entries in tests/common/plugins/conditional_mark/tests_mark_conditions.yaml for platform x86_64-nvidia_sn5640-r0 with reason explaining decap is not supported.

How did you verify/test it?

Verified the YAML syntax and confirmed the conditional mark system picks up the new entries.

Any platform specific information?

Applies to x86_64-nvidia_sn5640-r0 platform only.

Documentation

N/A

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Add xfail mark for decap/test_decap.py::test_decap and
vxlan/test_vnet_decap.py::test_vnet_decap on x86_64-nvidia_sn5640-r0
platform.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Ze Gan <ganze718@gmail.com>
@Pterosaur Pterosaur force-pushed the zegan/xfail_sn5640_decap_tests branch from e7f2ff1 to d963a29 Compare March 25, 2026 14:19
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Collaborator

@yxieca yxieca left a comment

Choose a reason for hiding this comment

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

AI agent on behalf of Ying. Reviewed; no issues found.

Pterosaur added a commit to Pterosaur/sonic-mgmt that referenced this pull request Mar 26, 2026
…t_decap on SN5640

Add xfail mark for decap/test_decap.py::test_decap and vxlan/test_vnet_decap.py::test_vnet_decap on x86_64-nvidia_sn5640-r0 platform.

Upstream PR: sonic-net#23302

----
#### AI description  (iteration 1)
#### PR Classification
Test update: modifying test conditions to mark expected failures on the SN5640 platform.

#### PR Summary
This pull request adjusts test configurations to avoid false failures on the x86_64-nvidia_sn5640-r0 platform by marking two tests as xfail.
- `tests/common/plugins/conditional_mark/tests_mark_conditions.yaml`: Added an xfail condition for `vxlan/test_vnet_decap.py::test_vnet_decap` with appropriate platform criteria.
- `decap/test_decap.py`: Introduced an xfail block for test cases, applying the same platform condition for expected failures.
<!-- GitOpsUserAgent=GitOps.Apps.Server.pullrequestcopilot -->
@yxieca yxieca merged commit 9c15289 into sonic-net:master Mar 26, 2026
21 checks passed
ravaliyel pushed a commit to ravaliyel/sonic-mgmt that referenced this pull request Mar 27, 2026
…sonic-net#23302)

What is the motivation for this PR?\n and  consistently fail on  because decap is not supported on this platform.\n\nHow did you do it?\nAdded xfail entries in tests/common/plugins/conditional_mark/tests_mark_conditions.yaml for platform  with reason explaining decap is not supported.\n\nHow did you verify/test it?\nVerified the YAML syntax and confirmed the conditional mark system picks up the new entries.\n\nSigned-off-by:\nZe Gan <ganze718@gmail.com>
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