Skip to content

zebra: fix import of non zebra extern_learn neighbors #19122

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

louis-6wind
Copy link
Contributor

"extern_learn" neighbor entries not created by zebra are not imported into EVPN, because zebra incorrectly assumes it originated them. For example:

192.168.0.1 dev br100 lladdr de:ed:01:d0:52:5f extern_learn REACHABLE

In contrast, entries created by zebra are explicitly marked with proto zebra:

192.168.0.2 dev br100 lladdr de:ed:01:dc:0e:62 extern_learn NOARP proto zebra

Only consider an entry as its own if it is marked with both extern_learn and proto zebra.

"extern_learn" neighbor entries not created by zebra are not imported
into EVPN, because zebra incorrectly assumes it originated them. For
example:

> 192.168.0.1 dev br100 lladdr de:ed:01:d0:52:5f extern_learn REACHABLE

In contrast, entries created by zebra are explicitly marked with proto zebra:

> 192.168.0.2 dev br100 lladdr de:ed:01:dc:0e:62 extern_learn NOARP proto zebra

Only consider an entry as its own if it is marked with both extern_learn
and proto zebra.

Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
@louis-6wind
Copy link
Contributor Author

ci:rerun

@donaldsharp
Copy link
Member

I think this needs a topology test to show that this works as expected.

@github-actions github-actions bot added the rebase PR needs rebase label Jul 9, 2025
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.

3 participants