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

bgpd: add bgp ipv6-auto-ra command (backport #16354) #17270

Closed
wants to merge 1 commit into from

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Oct 28, 2024

Introduce a command to completely disable sending IPv6 router advertisement messages on an interface. Before, there were cases where it could be enabled by other daemons, namely bgpd and vrrpd. Particularly, it happens when BGP extended-nexthop capability is used: bgpd tells zebra to enable RA in this case, though it wasn't intended.

Fixes #7738.


This is an automatic backport of pull request #16354 done by [Mergify](https://mergify.com).

Introduce a command to stop bgpd from enabling IPv6 router advertisement
messages sending on interfaces.

Signed-off-by: Mikhail Sokolovskiy <sokolmish@gmail.com>
(cherry picked from commit 7b1c0c2)

# Conflicts:
#	bgpd/bgp_nht.c
#	bgpd/bgp_vty.c
#	bgpd/bgpd.c
#	bgpd/bgpd.h
@mergify mergify bot added the conflicts label Oct 28, 2024
Copy link
Author

mergify bot commented Oct 28, 2024

Cherry-pick of 7b1c0c2 has failed:

On branch mergify/bp/stable/8.5/pr-16354
Your branch is up to date with 'origin/stable/8.5'.

You are currently cherry-picking commit 7b1c0c23fc.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   bgpd/bgp_zebra.c
	modified:   doc/user/bgp.rst
	modified:   doc/user/ipv6.rst

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   bgpd/bgp_nht.c
	both modified:   bgpd/bgp_vty.c
	both modified:   bgpd/bgpd.c
	both modified:   bgpd/bgpd.h

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

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