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.

Sign up for GitHub

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

Jump to bottom

[action] [PR:16367] Assign the higher metric value for Ipv6 default route learnt via RA message #16429

Merged
merged 1 commit into from
Sep 3, 2023
Merged

[action] [PR:16367] Assign the higher metric value for Ipv6 default route learnt via RA message #16429

merged 1 commit into from
Sep 3, 2023

Conversation

mssonicbld
Copy link
Collaborator

What I did:
Assign the metric value for Ipv6 default route learnt via RA message to
higher value so that BGP learnt default route is higher priority.

This change depend on this Kernel Patch: sonic-net/sonic-linux-kernel#326

ADO: 24888962

Why I did:
Fixes : #16209

How I verify:
Manual Verification to make sure after setting metric e-BGP learnt default is always higher priority over RA default route.

str2-xxxxx-acs-03# show ipv6 route ::/0
Routing entry for ::/0
 Known via "kernel", distance 119, metric 1000
 Last update 00:00:33 ago
 fe80::9a49:25ff:fe03:6e52, via eth0

Routing entry for ::/0
 Known via "bgp", distance 20, metric 0, best
 Last update 00:00:59 ago
 LICENSE MAINTAINERS Makefile Makefile.cache Makefile.work README.buildsystem.md README.md ThirdPartyLicenses.txt azure-pipelines.yml build_debian.sh build_debug_docker_j2.sh build_docker.sh build_image.sh check_install.py device dockers files functions.sh get_docker-base.sh install_sonic.py installer onie-image-arm64.conf onie-image-armhf.conf onie-image.conf onie-mk-demo.sh platform push_docker.sh rules scripts slave.mk sonic-slave-bullseye sonic-slave-buster sonic-slave-jessie sonic-slave-stretch src update_screen.sh fc00::2, via PortChannel102, weight 1
 LICENSE MAINTAINERS Makefile Makefile.cache Makefile.work README.buildsystem.md README.md ThirdPartyLicenses.txt azure-pipelines.yml build_debian.sh build_debug_docker_j2.sh build_docker.sh build_image.sh check_install.py device dockers files functions.sh get_docker-base.sh install_sonic.py installer onie-image-arm64.conf onie-image-armhf.conf onie-image.conf onie-mk-demo.sh platform push_docker.sh rules scripts slave.mk sonic-slave-bullseye sonic-slave-buster sonic-slave-jessie sonic-slave-stretch src update_screen.sh fc00::a, via PortChannel105, weight 1
 LICENSE MAINTAINERS Makefile Makefile.cache Makefile.work README.buildsystem.md README.md ThirdPartyLicenses.txt azure-pipelines.yml build_debian.sh build_debug_docker_j2.sh build_docker.sh build_image.sh check_install.py device dockers files functions.sh get_docker-base.sh install_sonic.py installer onie-image-arm64.conf onie-image-armhf.conf onie-image.conf onie-mk-demo.sh platform push_docker.sh rules scripts slave.mk sonic-slave-bullseye sonic-slave-buster sonic-slave-jessie sonic-slave-stretch src update_screen.sh fc00::12, via PortChannel108, weight 1
 LICENSE MAINTAINERS Makefile Makefile.cache Makefile.work README.buildsystem.md README.md ThirdPartyLicenses.txt azure-pipelines.yml build_debian.sh build_debug_docker_j2.sh build_docker.sh build_image.sh check_install.py device dockers files functions.sh get_docker-base.sh install_sonic.py installer onie-image-arm64.conf onie-image-armhf.conf onie-image.conf onie-mk-demo.sh platform push_docker.sh rules scripts slave.mk sonic-slave-bullseye sonic-slave-buster sonic-slave-jessie sonic-slave-stretch src update_screen.sh fc00::1a, via PortChannel1011, weight 1
 LICENSE MAINTAINERS Makefile Makefile.cache Makefile.work README.buildsystem.md README.md ThirdPartyLicenses.txt azure-pipelines.yml build_debian.sh build_debug_docker_j2.sh build_docker.sh build_image.sh check_install.py device dockers files functions.sh get_docker-base.sh install_sonic.py installer onie-image-arm64.conf onie-image-armhf.conf onie-image.conf onie-mk-demo.sh platform push_docker.sh rules scripts slave.mk sonic-slave-bullseye sonic-slave-buster sonic-slave-jessie sonic-slave-stretch src update_screen.sh fc00::22, via PortChannel1014, weight 1
 LICENSE MAINTAINERS Makefile Makefile.cache Makefile.work README.buildsystem.md README.md ThirdPartyLicenses.txt azure-pipelines.yml build_debian.sh build_debug_docker_j2.sh build_docker.sh build_image.sh check_install.py device dockers files functions.sh get_docker-base.sh install_sonic.py installer onie-image-arm64.conf onie-image-armhf.conf onie-image.conf onie-mk-demo.sh platform push_docker.sh rules scripts slave.mk sonic-slave-bullseye sonic-slave-buster sonic-slave-jessie sonic-slave-stretch src update_screen.sh fc00::2a, via PortChannel1017, weight 1
 LICENSE MAINTAINERS Makefile Makefile.cache Makefile.work README.buildsystem.md README.md ThirdPartyLicenses.txt azure-pipelines.yml build_debian.sh build_debug_docker_j2.sh build_docker.sh build_image.sh check_install.py device dockers files functions.sh get_docker-base.sh install_sonic.py installer onie-image-arm64.conf onie-image-armhf.conf onie-image.conf onie-mk-demo.sh platform push_docker.sh rules scripts slave.mk sonic-slave-bullseye sonic-slave-buster sonic-slave-jessie sonic-slave-stretch src update_screen.sh fc00::32, via PortChannel1020, weight 1
 LICENSE MAINTAINERS Makefile Makefile.cache Makefile.work README.buildsystem.md README.md ThirdPartyLicenses.txt azure-pipelines.yml build_debian.sh build_debug_docker_j2.sh build_docker.sh build_image.sh check_install.py device dockers files functions.sh get_docker-base.sh install_sonic.py installer onie-image-arm64.conf onie-image-armhf.conf onie-image.conf onie-mk-demo.sh platform push_docker.sh rules scripts slave.mk sonic-slave-bullseye sonic-slave-buster sonic-slave-jessie sonic-slave-stretch src update_screen.sh fc00::3a, via PortChannel1023, weight 1

admin@str2-xxxxx-acs-03:~$ sudo ip -6 route show ::/0 table main
default nhid 297 proto bgp src fc00:1::32 metric 20 pref medium
 nexthop via fc00::2 dev PortChannel102 weight 1
 nexthop via fc00::12 dev PortChannel108 weight 1
 nexthop via fc00::1a dev PortChannel1011 weight 1
 nexthop via fc00::22 dev PortChannel1014 weight 1
 nexthop via fc00::2a dev PortChannel1017 weight 1
 nexthop via fc00::32 dev PortChannel1020 weight 1
 nexthop via fc00::3a dev PortChannel1023 weight 1
 nexthop via fc00::a dev PortChannel105 weight 1
default via fe80::2a01:111:e210:3000:3 dev eth0 proto ra metric 1996489704 expires 1797sec hoplimit 64 pref medium
default via fe80::2a01:111:e210:3000:1 dev eth0 proto ra metric 1996489704 expires 1799sec hoplimit 64 pref medium
default via fe80::2a01:111:e210:3000:2 dev eth0 proto ra metric 1996489704 expires 1797sec hoplimit 64 pref medium
default via fe80::9a49:25ff:fe03:6e52 dev eth0 proto ra metric 1996489704 expires 1797sec hoplimit 64 pref medium

@abdosi @mssonicbld
Assign the higher metric value for Ipv6 default route learnt via RA m…
b9cfb3e 
…essage (sonic-net#16367)

* Fix the Loopback0 IPv6 address of LC's in chassis not reachable from peer device's
* Assign the metric vaule for Ipv6 default route learnt via RA message to higher value so that BGP learnt default route is higher priority.

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
@mssonicbld mssonicbld requested a review from lguohan as a code owner September 3, 2023 03:36
@mssonicbld mssonicbld mentioned this pull request Sep 3, 2023
Merged
@mssonicbld
Copy link
Collaborator Author

Original PR: #16367

@mssonicbld mssonicbld merged commit 40a5cea into sonic-net:202305 Sep 3, 2023
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lguohan lguohan Awaiting requested review from lguohan lguohan is a code owner

Assignees
No one assigned
Labels
automerge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mssonicbld @abdosi