Skip to content

Skip modular chassis specific tests on disaggregated chassis#18384

Open
tjchadaga wants to merge 4 commits intosonic-net:masterfrom
tjchadaga:skip_chassis_tests
Open

Skip modular chassis specific tests on disaggregated chassis#18384
tjchadaga wants to merge 4 commits intosonic-net:masterfrom
tjchadaga:skip_chassis_tests

Conversation

@tjchadaga
Copy link
Contributor

Description of PR

Summary:
Fixes # (issue)
Skip modular chassis tests on disaggregated chassis

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?

Tests specific for modular chassis are not applicable for disaggregated chassis. These are skipped in the PR

How did you do it?

Check topo name to identify disaggregated chassis and skip the tests

How did you verify/test it?

Ran both tests on disaggregated chassis to confirm skip

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@tjchadaga tjchadaga requested review from YatishSVC and arlakshm May 13, 2025 20:28
YatishSVC
YatishSVC previously approved these changes May 13, 2025
Copy link
Contributor

@YatishSVC YatishSVC left a comment

Choose a reason for hiding this comment

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

lgtm

arlakshm
arlakshm previously approved these changes May 17, 2025
@arlakshm
Copy link
Contributor

/Azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

arlakshm pushed a commit to Azure/sonic-mgmt.msft that referenced this pull request May 19, 2025
…ggregated chassis (#300)

Manual cherry-pick of sonic-net/sonic-mgmt#18384

What is the motivation for this PR?
Tests specific for modular chassis are not applicable for disaggregated
chassis. These are skipped in the PR

How did you do it?
Check topo name to identify disaggregated chassis and skip the tests

How did you verify/test it?
Ran both tests on disaggregated chassis to confirm skip
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.

4 participants