Skip to content

[Mellanox][202411] Update the skip for test test_qos_dscp_mapping.py#19381

Merged
bingwang-ms merged 1 commit intosonic-net:202411from
congh-nvidia:test_qos_dscp_mapping_202411
Jul 8, 2025
Merged

[Mellanox][202411] Update the skip for test test_qos_dscp_mapping.py#19381
bingwang-ms merged 1 commit intosonic-net:202411from
congh-nvidia:test_qos_dscp_mapping_202411

Conversation

@congh-nvidia
Copy link
Contributor

Description of PR

Summary:
In 202411, only uniform mode is supported by Mellanox platforms for non-dualtor testbeds.
For dualtor testbed, only pipe is supported to support the the dscp remapping.

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?

To correct the skip condition for test_qos_dscp_mapping.py and Mellanox platforms.

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
Collaborator

/azp run

@azure-pipelines
Copy link

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.

In 202411, only uniform mode is supported by Mellanox platforms for non-dualtor.
For dualtor, only pipe is supported due to the dscp remapping.
@congh-nvidia congh-nvidia force-pushed the test_qos_dscp_mapping_202411 branch from b65aaab to 8b52c91 Compare July 3, 2025 11:42
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@congh-nvidia
Copy link
Contributor Author

Hi @lolyu @bingwang-ms , could you please review?

Copy link
Collaborator

@lolyu lolyu 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 573d42d into sonic-net:202411 Jul 8, 2025
14 checks passed
@congh-nvidia congh-nvidia deleted the test_qos_dscp_mapping_202411 branch July 9, 2025 06:24
sdszhang pushed a commit to sdszhang/sonic-mgmt that referenced this pull request Aug 2, 2025
Code sync sonic-net/sonic-mgmt:202411 => 202412

```
*   3124d52 (HEAD -> code-sync-202412, origin/code-sync-202412) r12f 250710:1626 - Merge remote-tracking branch 'base/202411' into code-sync-202412
|\
| * b1b95dc (base/202411) zitingguo-ms 250513:1848 - Fix PortChannel name matching in verify_attr_change to handle leading spaces (sonic-net#18301)
| * 7461bf3 wrideout-arista 250604:1542 - Determine if the eos switch is running in multiagent mode, and stop/start the (sonic-net#18748)
| * 4346afd Dashuai Zhang 250411:0103 - update d18u8s4 PT0 ASN to 4 bytes (sonic-net#17888)
| * de1b1b4 Ryangwaite 250709:1028 - Skip multi-hop upgrade tests on non-T0 testbeds (sonic-net#19078) (sonic-net#19326)
| * 573d42d Cong Hou 250709:0232 - Update the skip for test test_qos_dscp_mapping.py (sonic-net#19381)
| * 3eb65b9 Yawen 250708:1743 - Unskip test_qos_sai for dualtor-aa-64-breakout (sonic-net#19363)
| * f0c9705 Cong Hou 250707:2351 - xfail generic hash test on dualtor (sonic-net#19383)
| * d378110 eyakubch 250707:0546 - Reduce flakiness of test_l2_configure.py. (sonic-net#18831)
| * 043dba8 ShiyanWangMS 250703:0935 - Fix the ignore pattern - leapsecond file expired warning (sonic-net#19332)
| * 92af38f zitingguo-ms 250703:1958 - Unskip testQosSaiHeadroomPoolWatermark on t0-d18u8s4 (sonic-net#19287) (sonic-net#19342)
| * 0b7e4de Chuan Wu 250704:0334 - Add background traffic for test_pfc_pause_extra_lossless test (sonic-net#19218)
```
congh-nvidia pushed a commit to congh-nvidia/sonic-mgmt that referenced this pull request Nov 20, 2025
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.

5 participants