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

[YANG] Add missing field in port qos map #21677

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

wen587
Copy link
Contributor

@wen587 wen587 commented Feb 8, 2025

Why I did it

Add missing field introduced from PR: #21395

Work item tracking
  • Microsoft ADO (number only): 31313829

How I did it

Add the missing field

How to verify it

UT in sample_config_db.json
Also test in specific DUT after merge this change.
before:

sonic_yang(3):All Keys are not parsed in PORT_QOS_MAP
dict_keys(['Ethernet16'])
sonic_yang(3):exceptionList:["'tc_to_dscp_map'"]

after:

sonic_yang(6):Note: Below table(s) have no YANG models: bgpraw

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@wen587 wen587 requested review from kperumalbfn and ganglyu February 8, 2025 05:33
@wen587 wen587 marked this pull request as ready for review February 8, 2025 05:41
@wen587 wen587 requested a review from qiluo-msft as a code owner February 8, 2025 05:41
@ganglyu
Copy link
Contributor

ganglyu commented Feb 8, 2025

Please add unit test

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

3 participants