Skip to content

disable pfcwd before running qos_sai testing.#17393

Merged
yejianquan merged 4 commits intosonic-net:masterfrom
sdszhang:disable_pfcwd
Mar 12, 2025
Merged

disable pfcwd before running qos_sai testing.#17393
yejianquan merged 4 commits intosonic-net:masterfrom
sdszhang:disable_pfcwd

Conversation

@sdszhang
Copy link
Copy Markdown
Contributor

@sdszhang sdszhang commented Mar 6, 2025

Description of PR

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

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?

TBD

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

@sdszhang
Copy link
Copy Markdown
Contributor Author

sdszhang commented Mar 6, 2025

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

yejianquan
yejianquan previously approved these changes Mar 7, 2025
Copy link
Copy Markdown
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, waiting for the test to pass

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Comment thread tests/common/plugins/conditional_mark/tests_mark_conditions.yaml
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

@sdszhang sdszhang closed this Mar 7, 2025
@sdszhang sdszhang reopened this Mar 7, 2025
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

yejianquan
yejianquan previously approved these changes Mar 7, 2025
Copy link
Copy Markdown
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

# Conflicts:
#	tests/qos/qos_sai_base.py
# Conflicts:
#	tests/common/plugins/conditional_mark/tests_mark_conditions.yaml
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
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 69034ab into sonic-net:master Mar 12, 2025
19 checks passed
@mssonicbld
Copy link
Copy Markdown
Collaborator

@sdszhang PR conflicts with 202411 branch

sdszhang added a commit to sdszhang/sonic-mgmt that referenced this pull request Mar 12, 2025
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: [email protected]

(cherry picked from commit 69034ab)
sdszhang added a commit to sdszhang/sonic-mgmt that referenced this pull request Mar 12, 2025
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: [email protected]

(cherry picked from commit 69034ab)
yejianquan pushed a commit that referenced this pull request Mar 14, 2025
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: [email protected]

(cherry picked from commit 69034ab)
nnelluri-cisco pushed a commit to nnelluri-cisco/sonic-mgmt that referenced this pull request Mar 15, 2025
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: [email protected]
@sdszhang sdszhang deleted the disable_pfcwd branch March 27, 2025 12:26
OriTrabelsi pushed a commit to OriTrabelsi/sonic-mgmt that referenced this pull request Apr 1, 2025
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: [email protected]
zhixzhu pushed a commit to zhixzhu/sonic-mgmt that referenced this pull request May 1, 2025
[PR:17393][202405] disable pfcwd before running qos_sai testing.

Cherry pick sonic-mgmt sonic-net#17393

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: [email protected]
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