Skip to content

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

Merged
lguohan merged 4 commits intosonic-net:masterfrom
wendani:dot1p_vs_master
Sep 13, 2019
Merged

Resubmit: use dot1p to tc mapping for backend switches#3422
lguohan merged 4 commits intosonic-net:masterfrom
wendani:dot1p_vs_master

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 wendani requested review from jleveque and lguohan September 9, 2019 22:34
@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