Skip to content

Conversation

@komal-marvell
Copy link
Contributor

@komal-marvell komal-marvell commented Sep 17, 2025

Qos map can be assigned to Router interface for Ingress mapping and Egress marking for DSCP and DOT1P

@eddyk-nvidia
Copy link

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 ?

@tjchadaga tjchadaga added the reviewed PR is discussed in SAI Meeting label Sep 22, 2025
@tjchadaga
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@tjchadaga
Copy link
Collaborator

@komal-marvell - could you please help address the comment?

@komal-marvell
Copy link
Contributor Author

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 ?

As discussed in community meeting, updated the PR with the precedence order. Please check in the header comment description.

@tjchadaga
Copy link
Collaborator

@eddyk-nvidia - could you please help sign-off on this?

@tjchadaga
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@tjchadaga
Copy link
Collaborator

@komal-marvell - please rebase your branch and squash commit to fix failures

Signed-off-by: Komal Shah <kshah2@marvell.com>
@komal-marvell
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 2208 in repo opencomputeproject/SAI

@tjchadaga
Copy link
Collaborator

/azp run

@azure-pipelines
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

Labels

reviewed PR is discussed in SAI Meeting

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants