Skip to content

[action] [PR:15870] Update all port pfcwd storm test to make it more stable#19088

Merged
mssonicbld merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/15870
Sep 3, 2025
Merged

[action] [PR:15870] Update all port pfcwd storm test to make it more stable#19088
mssonicbld merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/15870

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Description of PR

Update all port pfcwd storm test to make it more stable

  1. Optimize background traffic
  2. Fix arp resolve issue
  3. Remove ptf leftovers added in pfcwd test

Summary:
Fixes # (issue)
All port pfcwd storm test not stable enough.

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?

Update all port pfcwd storm test is not stable engough.

How did you do it?

Optimize background traffic and arp resolvation.

How did you verify/test it?

Run it in internal regression.

Any platform specific information?

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

Documentation

)

1. Optimize background traffic
2. Optimize pfcwd polling interval
3. Fix arp resolve issue
4. Remove ptf leftovers added in pfcwd test

Change-Id: I84ee67f56b4c65c682a175515737d3a9125cfdc7
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #15870

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@bingwang-ms
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-mgmt

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit a1a5691 into sonic-net:202411 Sep 3, 2025
14 checks passed
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