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

xZebra static blackhole nh #14605

Closed
wants to merge 2 commits into from

Conversation

bhinin
Copy link
Contributor

@bhinin bhinin commented Oct 16, 2023

No description provided.

bhinin and others added 2 commits September 28, 2023 10:44
Merge from upstream frr/master
In case a static route is resolving over multiple nexthops (multipath),
if the first nexthop in the list is recursively resolving over a
blackhole, zebra does not bother looking at other nexthops and sends a
netlink update to the kernel marking the route as a blackhole. This
results in complete traffic blackholing even in a multipath setup.

If number of nexthops > 1, do not bother checking for recursively
resolved blackhole nexthop. Let zebra's multipath handling take over.

Signed-off-by: Abhishek Naik <[email protected]>
@frrbot frrbot bot added the zebra label Oct 16, 2023
@bhinin bhinin closed this Oct 16, 2023
@github-actions github-actions bot added the rebase PR needs rebase label Oct 16, 2023
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.

1 participant