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

[Multi-asic] Fixed IPv6 Default Route to be BGP Learned #5548

Merged
merged 1 commit into from
Oct 6, 2020

Conversation

abdosi
Copy link
Contributor

@abdosi abdosi commented Oct 5, 2020

Why I did:
After PR #5418
Extending the changes done for v4 in #4774 to v6 now.

How I verify

Before:
sudo ip -6 -n asic0 route show ::/0
default via fd00::1 dev eth0 metric 1024 pref medium

After:
sudo ip -6 -n asic0 route show ::/0
default via fc00::2 dev PortChannel0002 proto 186 src fc00:1::32 metric 20 pref medium
default via fc00::6 dev PortChannel0005 proto 186 src fc00:1::32 metric 20 pref medium
default via fd00::1 dev eth0 metric 3523215360 pref medium

Learned and not docker default route for multi-asic platforms.

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
Copy link
Contributor

@arlakshm arlakshm left a comment

Choose a reason for hiding this comment

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

lgtm

@abdosi abdosi merged commit 70528f7 into sonic-net:master Oct 6, 2020
@abdosi abdosi deleted the namespace_default_gw branch October 6, 2020 05:54
abdosi added a commit that referenced this pull request Oct 6, 2020
Learned and not docker default route for multi-asic platforms.

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
santhosh-kt pushed a commit to santhosh-kt/sonic-buildimage that referenced this pull request Feb 25, 2021
Learned and not docker default route for multi-asic platforms.

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
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.

2 participants