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

[crm] Fix for Issue Azure/sonic-buildimage#8036 #1829

Merged
merged 1 commit into from
Aug 3, 2021

Conversation

qbdwlr
Copy link
Contributor

@qbdwlr qbdwlr commented Jul 20, 2021

What I did
Added more appropriate log message for CRM queries of MPLS_INSEG and MPLS_NEXTHOP resources when platform does not support these resource types.

Why I did it
Mellanox has raised the issue officially here: sonic-net/sonic-buildimage#8036

How I verified it
System test with VS SAI modified to return NOT_SUPPORTED.

Details if related
N/A

@qbdwlr qbdwlr requested a review from prsunny as a code owner July 20, 2021 13:28
@qbdwlr qbdwlr changed the title [crm] Fix for Issue sonic-buildimage/#8036 [crm] Fix for Issue Azure/sonic-buildimage#8036 Jul 20, 2021
@qbdwlr
Copy link
Contributor Author

qbdwlr commented Jul 20, 2021

@stepanblyschak @Junchao-Mellanox Please review this PR fix for sonic-net/sonic-buildimage#8036

@prsunny
Copy link
Collaborator

prsunny commented Jul 25, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny prsunny merged commit 72a72f8 into sonic-net:master Aug 3, 2021
qiluo-msft added a commit to qiluo-msft/sonic-swss that referenced this pull request Aug 4, 2021
judyjoseph pushed a commit that referenced this pull request Sep 2, 2021
*Added more appropriate log message for CRM queries of MPLS_INSEG and MPLS_NEXTHOP resources when platform does not support these resource types.
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-swss that referenced this pull request Oct 5, 2021
*Added more appropriate log message for CRM queries of MPLS_INSEG and MPLS_NEXTHOP resources when platform does not support these resource types.
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.

4 participants