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

Resubmit: use dot1p to tc mapping for backend switches #3422

Merged
merged 4 commits into from
Sep 13, 2019

Conversation

wendani
Copy link
Contributor

@wendani wendani commented Sep 9, 2019

Also deal with the situation when key 'type' is not defined in CONFIG_DB DEVICE_METADATA['localhost'], such as in the vs case

Signed-off-by: Wenda Ni [email protected]

- What I did

- How I did it

- How to verify it

- Description for the changelog

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

@wendani wendani changed the title vs test fix for dot1p to tc map where DEVICE_METADATA['localhost']['type'] is not defined vs test fix for dot1p to tc mapping when DEVICE_METADATA['localhost']['type'] is not defined Sep 9, 2019
@wendani wendani requested review from biblix and lguohan and removed request for biblix September 9, 2019 20:08
@wendani wendani changed the title vs test fix for dot1p to tc mapping when DEVICE_METADATA['localhost']['type'] is not defined dot1p to tc mapping fix for vs where DEVICE_METADATA['localhost']['type'] is not defined Sep 9, 2019
in DEVICE_METADATA['localhost'], such as in the vs case

Signed-off-by: Wenda Ni <[email protected]>
@wendani wendani changed the title dot1p to tc mapping fix for vs where DEVICE_METADATA['localhost']['type'] is not defined Use dot1p to tc mapping for backend switches Sep 9, 2019
@wendani wendani changed the title Use dot1p to tc mapping for backend switches Resubmit: use dot1p to tc mapping for backend switches Sep 9, 2019
@wendani
Copy link
Contributor Author

wendani commented Sep 10, 2019

🔍

@wendani
Copy link
Contributor Author

wendani commented Sep 10, 2019

🚢 🇮🇹

@lguohan lguohan merged commit 81aef6b into sonic-net:master Sep 13, 2019
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