Skip to content

[action] [PR:12733] Update the pfcwd test cases according to the new implement of the pfcwd.#13778

Merged
mssonicbld merged 1 commit intosonic-net:202311from
mssonicbld:cherry/202311/12733
Jul 23, 2024
Merged

[action] [PR:12733] Update the pfcwd test cases according to the new implement of the pfcwd.#13778
mssonicbld merged 1 commit intosonic-net:202311from
mssonicbld:cherry/202311/12733

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Update the pfcwd test cases according to the new implement of the pfcwd
After the feature is completed, it criteria to trigger the pfcwd is much more strict than previous on Nvidia platform, So adjust the test case to properly trigger the pfcwd. The solution is to send background traffic during the pfc frame is sent from the fanout.

Description of PR

Summary:
Fixes # (issue)

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205
  • 202305
  • 202311

Approach

What is the motivation for this PR?

To make the pfcwd test case could pass after the new implement of the pfcwd.

How did you do it?

Add the logic of sending the background traffic to make the test case could pass after the new implement of the pfcwd

How did you verify/test it?

All pfcwd test case could pass

Any platform specific information?

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

Documentation

…wd. (sonic-net#12733)

Update the pfcwd test cases according to the new implement of the pfcwd
After the feature is completed, it criteria to trigger the pfcwd is much more strict than previous on Nvidia platform, So adjust the test case to properly trigger the pfcwd. The solution is to send background traffic during the pfc frame is sent from the fanout.
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #12733

@mssonicbld mssonicbld merged commit bae9832 into sonic-net:202311 Jul 23, 2024
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.

2 participants