Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generate switch level dscp_to_tc_map entry from qos_config template #11087

Merged
merged 6 commits into from
Jun 17, 2022

Conversation

bingwang-ms
Copy link
Contributor

@bingwang-ms bingwang-ms commented Jun 9, 2022

Signed-off-by: bingwang wang.bing@microsoft.com

Why I did it

This PR is to update template for generating switch level DSCP_TO_TC_MAP config.
After PR #10565, there will be two DSCP_TO_TC_MAP

  • DSCP_TO_TC_MAP|AZURE is the default map, which is used at port level and switch level
  • DSCP_TO_TC_MAP|AZURE_TUNNEL is used to remap the priority of tunnel traffic in dualtor deployment

To address the issue, an entry PORT_QOS_MAP|global will be added into config_db

"PORT_QOS_MAP": {
        "global": {
            "dscp_to_tc_map": "AZURE"
        }
}

The entry is generated from qos_config.j2.

Related PRs

Repo PR title State
sonic-swss Apply DSCP_TO_TC_MAP from PORT_QOS_MAP|global to switch level GitHub issue/pull request detail
sonic-utilities Update db_migrator to support PORT_QOS_MAP|global GitHub issue/pull request detail
sonic-buildimage Update YANG for PORT_QOS_MAP to support switch level mapping GitHub issue/pull request detail

How I did it

Update the template qos_config.j2.

How to verify it

Verified by vstest.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

This PR is to update template for generating switch level DSCP_TO_TC_MAP config.

Link to config_db schema for YANG module changes

TODO

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: bingwang <wang.bing@microsoft.com>
@bingwang-ms bingwang-ms requested review from a team and lguohan as code owners June 9, 2022 08:02
@bingwang-ms bingwang-ms changed the title Generate switch level dscp_to_tc_map Generate switch level dscp_to_tc_map entry from qos_config.template Jun 9, 2022
@bingwang-ms bingwang-ms changed the title Generate switch level dscp_to_tc_map entry from qos_config.template Generate switch level dscp_to_tc_map entry from qos_config template Jun 9, 2022
Copy link
Contributor

@prsunny prsunny 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
Copy link
Contributor Author

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@bingwang-ms
Copy link
Contributor Author

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: bingwang <wang.bing@microsoft.com>
@bingwang-ms bingwang-ms merged commit 83f23e2 into sonic-net:master Jun 17, 2022
yxieca pushed a commit that referenced this pull request Jun 17, 2022
…11087)

* Generate switch level dscp_to_tc_map

Signed-off-by: bingwang <wang.bing@microsoft.com>
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