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

Set preference for forced mgmt routes #5844

Merged
merged 3 commits into from
Nov 10, 2020
Merged

Conversation

prsunny
Copy link
Contributor

@prsunny prsunny commented Nov 6, 2020

- Why I did it
When forced mgmt routes are present, the issue fixed as part of #5754 is not complete.

- How I did it
Added a preference(priority) field to forced mgmt route ip rules

- How to verify it
Verified on device

- Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@prsunny prsunny requested a review from lguohan November 6, 2020 20:28
lguohan
lguohan previously approved these changes Nov 6, 2020
@lguohan
Copy link
Collaborator

lguohan commented Nov 7, 2020

config gen test failing.

@prsunny
Copy link
Contributor Author

prsunny commented Nov 10, 2020

retest broadcom please

@lguohan
Copy link
Collaborator

lguohan commented Nov 10, 2020

retest broadcom please

@prsunny prsunny merged commit 1eaaf64 into sonic-net:master Nov 10, 2020
abdosi pushed a commit that referenced this pull request Nov 21, 2020
When forced mgmt routes are present, the issue fixed as part of #5754 is not complete.
Added a preference(priority) field to forced mgmt route ip rules
santhosh-kt pushed a commit to santhosh-kt/sonic-buildimage that referenced this pull request Feb 25, 2021
When forced mgmt routes are present, the issue fixed as part of sonic-net#5754 is not complete. 
Added a preference(priority) field to forced mgmt route ip rules
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.

3 participants