Skip to content

[action] [PR:17860] Specify 1-1 default map for TC_TO_DSCP for Cisco-8000#17948

Merged
mssonicbld merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/17860
Apr 11, 2025
Merged

[action] [PR:17860] Specify 1-1 default map for TC_TO_DSCP for Cisco-8000#17948
mssonicbld merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/17860

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Description of PR

Summary:
TC_TO_DSCP map defaults to 1-1 to help preserve encapsulated packet DSCPs during T1-transit. Reflect the default in the test configuration.
Fixes a couple tests that would fail to find some extra configuration when producing a DSCP->DSCP map.

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?

How did you do it?

How did you verify/test it?

Validated 2 failing tests in qos/test_tunnel_qos_remap.py passed on Cisco-8000 dualtor:

  • test_encap_dscp_rewrite
  • test_tunnel_decap_dscp_to_pg_mapping

Any platform specific information?

Specific to Cisco-8000 dualtor platform.

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

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #17860

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit 408bf9e into sonic-net:202411 Apr 11, 2025
14 checks passed
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.

2 participants