-
Notifications
You must be signed in to change notification settings - Fork 552
Support Qos Map on Router Interface #2208
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
base: master
Are you sure you want to change the base?
Conversation
|
As discussed during the community review please add some information about expected precedence order in case of mapping configured at different objects. Or it can be vendor specific ? |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@komal-marvell - could you please help address the comment? |
834c83a to
a58bd7b
Compare
As discussed in community meeting, updated the PR with the precedence order. Please check in the header comment description. |
|
@eddyk-nvidia - could you please help sign-off on this? |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@komal-marvell - please rebase your branch and squash commit to fix failures |
Signed-off-by: Komal Shah <kshah2@marvell.com>
a58bd7b to
62da084
Compare
|
/azp run |
|
Commenter does not have sufficient privileges for PR 2208 in repo opencomputeproject/SAI |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@eddyk-nvidia - could you please help complete the review? |
Qos map can be assigned to Router interface for Ingress mapping and Egress marking for DSCP and DOT1P