Skip to content

Enable new tests for Nvidia smartswitch#19275

Merged
bingwang-ms merged 2 commits intosonic-net:masterfrom
prabhataravind:paravind/ss_fixes
Jul 3, 2025
Merged

Enable new tests for Nvidia smartswitch#19275
bingwang-ms merged 2 commits intosonic-net:masterfrom
prabhataravind:paravind/ss_fixes

Conversation

@prabhataravind
Copy link
Copy Markdown
Contributor

  • There are tests that are skipped on these topologies when they shouldn't be.

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

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

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

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).

@prabhataravind prabhataravind marked this pull request as ready for review June 30, 2025 23:23
@prabhataravind prabhataravind changed the title Add t1-28-lag and t1-48-lag to tests_mark_conditions.yaml Add t1-48-lag to tests_mark_conditions.yaml Jul 1, 2025
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

bingwang-ms
bingwang-ms previously approved these changes Jul 2, 2025
* There are tests that are skipped on these topologies when they shouldn't be.

Signed-off-by: Prabhat Aravind <[email protected]>
 * Add t1-48-lag topo to the supported t1 topo list
 * Add Nvidia smartswitch hwskus to common/mellanox_data.py

Signed-off-by: Prabhat Aravind <[email protected]>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@prabhataravind prabhataravind changed the title Add t1-48-lag to tests_mark_conditions.yaml Enable new tests for Nvidia smartswitch Jul 3, 2025
Copy link
Copy Markdown
Contributor

@congh-nvidia congh-nvidia left a comment

Choose a reason for hiding this comment

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

LGTM

@bingwang-ms bingwang-ms merged commit a043e35 into sonic-net:master Jul 3, 2025
18 checks passed
@mssonicbld
Copy link
Copy Markdown
Collaborator

@prabhataravind PR conflicts with 202505 branch

prabhataravind added a commit to prabhataravind/sonic-mgmt that referenced this pull request Aug 3, 2025
Back port of sonic-net#19275 to 202505.

 * Add t1-48-lag topo to the supported t1 topo list
 * Add Nvidia smartswitch hwskus to common/mellanox_data.py

Signed-off-by: Prabhat Aravind <[email protected]>
prabhataravind added a commit to prabhataravind/sonic-mgmt that referenced this pull request Aug 3, 2025
Cherry-pick changes in PR sonic-net#19275 to 202505.

* There are tests that are skipped on these topologies when they shouldn't be.

Signed-off-by: Prabhat Aravind <[email protected]>
yejianquan pushed a commit that referenced this pull request Aug 4, 2025
Cherry-pick changes in PR #19275 to 202505.

* There are tests that are skipped on these topologies when they shouldn't be.

Signed-off-by: Prabhat Aravind <[email protected]>
nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Aug 7, 2025
* Add t1-48-lag to tests_mark_conditions.yaml

* There are tests that are skipped on these topologies when they shouldn't be.

Signed-off-by: Prabhat Aravind <[email protected]>

* Enable qos tests for Nvidia smartswitch

 * Add t1-48-lag topo to the supported t1 topo list
 * Add Nvidia smartswitch hwskus to common/mellanox_data.py

Signed-off-by: Prabhat Aravind <[email protected]>

---------

Signed-off-by: Prabhat Aravind <[email protected]>
ashutosh-agrawal pushed a commit to ashutosh-agrawal/sonic-mgmt that referenced this pull request Aug 14, 2025
* Add t1-48-lag to tests_mark_conditions.yaml

* There are tests that are skipped on these topologies when they shouldn't be.

Signed-off-by: Prabhat Aravind <[email protected]>

* Enable qos tests for Nvidia smartswitch

 * Add t1-48-lag topo to the supported t1 topo list
 * Add Nvidia smartswitch hwskus to common/mellanox_data.py

Signed-off-by: Prabhat Aravind <[email protected]>

---------

Signed-off-by: Prabhat Aravind <[email protected]>
vidyac86 pushed a commit to vidyac86/sonic-mgmt that referenced this pull request Oct 23, 2025
* Add t1-48-lag to tests_mark_conditions.yaml

* There are tests that are skipped on these topologies when they shouldn't be.

Signed-off-by: Prabhat Aravind <[email protected]>

* Enable qos tests for Nvidia smartswitch

 * Add t1-48-lag topo to the supported t1 topo list
 * Add Nvidia smartswitch hwskus to common/mellanox_data.py

Signed-off-by: Prabhat Aravind <[email protected]>

---------

Signed-off-by: Prabhat Aravind <[email protected]>
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
* Add t1-48-lag to tests_mark_conditions.yaml

* There are tests that are skipped on these topologies when they shouldn't be.

Signed-off-by: Prabhat Aravind <[email protected]>

* Enable qos tests for Nvidia smartswitch

 * Add t1-48-lag topo to the supported t1 topo list
 * Add Nvidia smartswitch hwskus to common/mellanox_data.py

Signed-off-by: Prabhat Aravind <[email protected]>

---------

Signed-off-by: Prabhat Aravind <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
* Add t1-48-lag to tests_mark_conditions.yaml

* There are tests that are skipped on these topologies when they shouldn't be.

Signed-off-by: Prabhat Aravind <[email protected]>

* Enable qos tests for Nvidia smartswitch

 * Add t1-48-lag topo to the supported t1 topo list
 * Add Nvidia smartswitch hwskus to common/mellanox_data.py

Signed-off-by: Prabhat Aravind <[email protected]>

---------

Signed-off-by: Prabhat Aravind <[email protected]>
Signed-off-by: Guy Shemesh <[email protected]>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
* Add t1-48-lag to tests_mark_conditions.yaml

* There are tests that are skipped on these topologies when they shouldn't be.

Signed-off-by: Prabhat Aravind <[email protected]>

* Enable qos tests for Nvidia smartswitch

 * Add t1-48-lag topo to the supported t1 topo list
 * Add Nvidia smartswitch hwskus to common/mellanox_data.py

Signed-off-by: Prabhat Aravind <[email protected]>

---------

Signed-off-by: Prabhat Aravind <[email protected]>
Signed-off-by: Aharon Malkin <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
* Add t1-48-lag to tests_mark_conditions.yaml

* There are tests that are skipped on these topologies when they shouldn't be.

Signed-off-by: Prabhat Aravind <[email protected]>

* Enable qos tests for Nvidia smartswitch

 * Add t1-48-lag topo to the supported t1 topo list
 * Add Nvidia smartswitch hwskus to common/mellanox_data.py

Signed-off-by: Prabhat Aravind <[email protected]>

---------

Signed-off-by: Prabhat Aravind <[email protected]>
Signed-off-by: Guy Shemesh <[email protected]>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
* Add t1-48-lag to tests_mark_conditions.yaml

* There are tests that are skipped on these topologies when they shouldn't be.

Signed-off-by: Prabhat Aravind <[email protected]>

* Enable qos tests for Nvidia smartswitch

 * Add t1-48-lag topo to the supported t1 topo list
 * Add Nvidia smartswitch hwskus to common/mellanox_data.py

Signed-off-by: Prabhat Aravind <[email protected]>

---------

Signed-off-by: Prabhat Aravind <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
* Add t1-48-lag to tests_mark_conditions.yaml

* There are tests that are skipped on these topologies when they shouldn't be.

Signed-off-by: Prabhat Aravind <[email protected]>

* Enable qos tests for Nvidia smartswitch

 * Add t1-48-lag topo to the supported t1 topo list
 * Add Nvidia smartswitch hwskus to common/mellanox_data.py

Signed-off-by: Prabhat Aravind <[email protected]>

---------

Signed-off-by: Prabhat Aravind <[email protected]>
Signed-off-by: Guy Shemesh <[email protected]>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
* Add t1-48-lag to tests_mark_conditions.yaml

* There are tests that are skipped on these topologies when they shouldn't be.

Signed-off-by: Prabhat Aravind <[email protected]>

* Enable qos tests for Nvidia smartswitch

 * Add t1-48-lag topo to the supported t1 topo list
 * Add Nvidia smartswitch hwskus to common/mellanox_data.py

Signed-off-by: Prabhat Aravind <[email protected]>

---------

Signed-off-by: Prabhat Aravind <[email protected]>
Signed-off-by: Yael Tzur <[email protected]>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
* Add t1-48-lag to tests_mark_conditions.yaml

* There are tests that are skipped on these topologies when they shouldn't be.

Signed-off-by: Prabhat Aravind <[email protected]>

* Enable qos tests for Nvidia smartswitch

 * Add t1-48-lag topo to the supported t1 topo list
 * Add Nvidia smartswitch hwskus to common/mellanox_data.py

Signed-off-by: Prabhat Aravind <[email protected]>

---------

Signed-off-by: Prabhat Aravind <[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.

5 participants