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

[202211]Custom monitoring based priority tunnels #2738

Conversation

siqbal1986
Copy link
Contributor

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

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]>
@siqbal1986 siqbal1986 requested a review from prsunny as a code owner April 17, 2023 21:40
Copy link
Collaborator

@prsunny prsunny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@siqbal1986
Copy link
Contributor Author

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.

@siqbal1986
Copy link
Contributor Author

Cannot merge until sonic-net/sonic-swss-common#725 cast to 202211 goes in. otherwise it will cause build failure in the branch.

@siqbal1986
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@siqbal1986
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@siqbal1986
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@siqbal1986 siqbal1986 closed this Apr 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants