Skip to content

Skip test_show_chassis_modules.py on T2 single node#19199

Merged
judyjoseph merged 2 commits intosonic-net:masterfrom
patrickmacarthur:master-skip-test-show-chassis-modules
Jul 25, 2025
Merged

Skip test_show_chassis_modules.py on T2 single node#19199
judyjoseph merged 2 commits intosonic-net:masterfrom
patrickmacarthur:master-skip-test-show-chassis-modules

Conversation

@patrickmacarthur
Copy link
Copy Markdown
Contributor

T2 single node does not run chassisd and thus does not fill in the chassis module table.

Description of PR

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Approach

What is the motivation for this PR?

T2 single node does not run chassisd and thus does not fill in the chassis module table.

How did you do it?

I added the test to the conditional skip list.

How did you verify/test it?

I verified that the test was skipped on a t2-single-node-min topology.

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 successfully started running 1 pipeline(s).

T2 single node does not run chassisd and thus does not fill in the
chassis module table.
@patrickmacarthur patrickmacarthur force-pushed the master-skip-test-show-chassis-modules branch from dbb8d98 to 897f36c Compare July 2, 2025 17:19
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@judyjoseph judyjoseph enabled auto-merge (squash) July 25, 2025 18:21
@judyjoseph
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@judyjoseph judyjoseph merged commit f48a05d into sonic-net:master Jul 25, 2025
18 of 19 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jul 28, 2025
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202505: #19861

nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Aug 7, 2025
ashutosh-agrawal pushed a commit to ashutosh-agrawal/sonic-mgmt that referenced this pull request Aug 14, 2025
vidyac86 pushed a commit to vidyac86/sonic-mgmt that referenced this pull request Oct 23, 2025
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

9 participants