-
Notifications
You must be signed in to change notification settings - Fork 558
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
[202211]Custom monitoring based priority tunnels #2738
[202211]Custom monitoring based priority tunnels #2738
Conversation
This PR implements the Vxlan ECMP priority tunnels where route next hop groups are switched based on the state of the next hop monitor session and their priority. This implementation allows primary and secondary next-hop groups which are switched based on the monitor state of the next hops. Please see the included tests to see what this code would do for detail. Linked-PR: sonic-net#2671 Signed-off-by: siqbal1486 <[email protected]>
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.
lgtm
it seems, this PR requires sonic-net/sonic-swss-common#725 CP to 202211. Requested initiated for CP. ill merge this once that request is completed. |
Cannot merge until sonic-net/sonic-swss-common#725 cast to 202211 goes in. otherwise it will cause build failure in the branch. |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
This PR implements the Vxlan ECMP priority tunnels where route next hop groups are switched based on the state of the next hop monitor session and their priority. This implementation allows primary and secondary next-hop groups which are switched based on the monitor state of the next hops. Please see the included tests to see what this code would do for detail.
cherry-pick #2719
What I did
Why I did it
How I verified it
Details if related