Skip to content

[202411] disable pfcwd before running qos_sai testing. (#17393)#17486

Merged
yejianquan merged 1 commit intosonic-net:202411from
sdszhang:cherry/202411/17393
Mar 14, 2025
Merged

[202411] disable pfcwd before running qos_sai testing. (#17393)#17486
yejianquan merged 1 commit intosonic-net:202411from
sdszhang:cherry/202411/17393

Conversation

@sdszhang
Copy link
Contributor

Description of PR
Summary:
After sonic-net/sonic-utilities#3792, we need to disable pfcwd on backplane ports.

co-authorized by: jianquanye@microsoft.com

(cherry picked from commit 69034ab)

Description of PR

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

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405
  • 202411

Approach

What is the motivation for this PR?

PFCWD needs to be disabled on backplane ports when running qos_sai testing

How did you do it?

added module level fixture to disable pfcwd

How did you verify/test it?

Any platform specific information?

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

Documentation

Description of PR
Summary:
After sonic-net/sonic-utilities#3792, we need to disable pfcwd on backplane ports.

Approach
What is the motivation for this PR?
PFCWD needs to be disabled on backplane ports when running qos_sai testing

How did you do it?
added module level fixture to disable pfcwd

co-authorized by: jianquanye@microsoft.com

(cherry picked from commit 69034ab)
@mssonicbld
Copy link
Collaborator

/azp run

@sdszhang sdszhang changed the title disable pfcwd before running qos_sai testing. (#17393) [202411] disable pfcwd before running qos_sai testing. (#17393) Mar 12, 2025
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Collaborator

@yejianquan yejianquan left a comment

Choose a reason for hiding this comment

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

LGTM

@yejianquan yejianquan merged commit 75e62ca into sonic-net:202411 Mar 14, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants