-
Notifications
You must be signed in to change notification settings - Fork 274
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
sonic-sairedis updates for MPLS #824
Conversation
@kcudnik I will need you to approve this PR yet again since the rest of the MPLS PRs are thisclose to approval, but this PR seems to have perpetual and unrelated issues with the Azure pipeline. What can I do about this? |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
@kcudnik Finally got a good pipeline run. Can you approve/merge this now? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please see comments
What I did
Minor tweaks to VS for MPLS support for CRM polling of MPLS In-segments and NHs.
Why I did it
Allows MPLS CRM support to function properly with VS.
How I verified it
Unit-tests in sonic-swss/tests/test_mpls.py.
Details if related
Refer to HLD: sonic-net/SONiC#706