Skip to content

xfail pfcwd test_pfcwd_warm_reboot case for Arista and Mellanox#15943

Closed
lipxu wants to merge 2 commits intosonic-net:masterfrom
lipxu:20241205_publicMaster_xfail_pfcwdWarmreboot
Closed

xfail pfcwd test_pfcwd_warm_reboot case for Arista and Mellanox#15943
lipxu wants to merge 2 commits intosonic-net:masterfrom
lipxu:20241205_publicMaster_xfail_pfcwdWarmreboot

Conversation

@lipxu
Copy link
Copy Markdown
Contributor

@lipxu lipxu commented Dec 8, 2024

Description of PR

Summary:
Fixes # (issue)
29870655

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

Approach

What is the motivation for this PR?

the case is flaky

How did you do it?

XFAIL the case until the testbed environment limitations are fixed

How did you verify/test it?

Run locally

Any platform specific information?

Broadcom

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

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@lipxu lipxu requested a review from bingwang-ms January 9, 2025 08:45
@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@lipxu lipxu changed the title xfail pfcwd warm reboot for broadcom xfail pfcwd test_pfcwd_warm_reboot case for Arista and Mellanox Jan 9, 2025
@bingwang-ms
Copy link
Copy Markdown
Collaborator

Probably we can do this change internally if community has other mechanism to workaround the limitation.

@lipxu
Copy link
Copy Markdown
Contributor Author

lipxu commented Jan 12, 2025

Probably we can do this change internally if community has other mechanism to workaround the limitation.

Good point, thanks for your suggestion @bingwang-ms , I will create an internal PR to xfail this case during internal run, thanks

@lipxu lipxu marked this pull request as draft January 12, 2025 13:46
@lipxu lipxu closed this Jan 12, 2025
@lipxu
Copy link
Copy Markdown
Contributor Author

lipxu commented Jan 12, 2025

Xfail the case in internal branch.

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