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

Export zebra package and some symbols #2727

Closed
wants to merge 6 commits into from

Conversation

wenovus
Copy link
Contributor

@wenovus wenovus commented Oct 24, 2023

Use case is to allow a different Go process/thread to exchange routes with GoBGP via Zebra API. For example, a custom Go-based RIB manager.

@fujita
Copy link
Member

fujita commented Oct 25, 2023

You are actually implementing a custom RIB manager that can replace zebra?

@wenovus
Copy link
Contributor Author

wenovus commented Oct 25, 2023

You are actually implementing a custom RIB manager that can replace zebra?

Currently we're only doing this in a limited way in order to integrate GoBGP as the BGP speaker for OpenConfig's reference implementation: https://github.com/openconfig/lemming/blob/main/sysrib/zapi.go, sysrib is the custom RIB manager for lemming.

@fujita
Copy link
Member

fujita commented Oct 26, 2023

Understood. If the CIs succeed, I'll merge this.

@fujita
Copy link
Member

fujita commented Oct 28, 2023

@irino any comments?

@fujita
Copy link
Member

fujita commented Oct 31, 2023

@wenovus pushed the first four commits.
@irino this simply makes zebra package public so the zebra feature should work as before.

@fujita fujita closed this Oct 31, 2023
@irino
Copy link
Contributor

irino commented Nov 2, 2023

@fujita OK, no proble,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants