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

zebra:check DAD freeze action before notifying bgp #17737

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

chiragshah6
Copy link
Member

If Duplicate Address Detection action is freeze
(permanent or definite time means not warn only mode) then locally duplicate detected MAC delete notification is not require to inform,
instead ask BGP to sync previous remote MAC entry. In freeze case local MAC event is not known to BGP, instead BGP is pointing to remote VTEP for the MAC.

Signed-off-by: Karthikeya Venkat Muppalla kmuppalla@nvidia.com
Signed-off-by: Chirag Shah chirag@nvidia.com

If Duplicate Address Detection action is freeze
(permanent or definite time means not warn only mode)
then locally duplicate detected MAC delete notification
is not require to inform,
instead ask BGP to sync previous remote MAC entry.
In freeze case local MAC event is not known to BGP,
instead BGP is pointing to remote VTEP for the MAC.

Ticket: #3652383
Issue: 3652383

Signed-off-by: Chirag Shah <chirag@nvidia.com>
Copy link
Member

@ton31337 ton31337 left a comment

Choose a reason for hiding this comment

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

LGTM, but... (out of this PR scope) we don't have any documentation regarding DAD in EVPN... Would be possible someone from Nvidia to add the documentation?

@ton31337
Copy link
Member

@frrbot rereview

@chiragshah6
Copy link
Member Author

LGTM, but... (out of this PR scope) we don't have any documentation regarding DAD in EVPN... Would be possible someone from Nvidia to add the documentation?

@ton31337 I will have small write up in subsequent PR

@ton31337 ton31337 merged commit 73fad72 into FRRouting:master Jan 3, 2025
11 checks passed
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