-
-
Notifications
You must be signed in to change notification settings - Fork 218
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
No-leader-election mode for BGP and fixes for routing table mode #740
No-leader-election mode for BGP and fixes for routing table mode #740
Conversation
Signed-off-by: Patryk Strusiewicz-Surmacki <patryk-pawel.strusiewicz-surmacki@external.telekom.de>
Signed-off-by: Patryk Strusiewicz-Surmacki <patryk-pawel.strusiewicz-surmacki@external.telekom.de>
pkg/manager/watch_endpointslices.go
Outdated
|
||
case watch.Deleted: | ||
if !sm.config.EnableServicesElection && !sm.config.EnableLeaderElection && sm.config.EnableRoutingTable { | ||
// When no-leade-elecition mode |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// When no-leade-elecition mode
-> // When no-leader-election mode
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
Looking good! Much left that needs adding? |
Any ETA on this @p-strusiewiczsurmacki-mobica ? |
Hi @thebsdbox, I'm waiting for @Cellebyte to test it with some real-life setup. I think we should be able to get this done unitl next week max. |
Signed-off-by: Patryk Strusiewicz-Surmacki <patryk-pawel.strusiewicz-surmacki@external.telekom.de>
@thebsdbox and @p-strusiewiczsurmacki-mobica I will build a new image and start testing ;) |
So from my testing.
I did not test again the deletion case which was already working as expected in the already merged PR. |
Looks good, going to merge this and create a new release! 🎉 |
This PR should fix #729
Added support for non-election-mode for BGP + some fixes for Service Policy in non-leader-election mode in routing table mode.
Additionally fixed some issues I found in BGP mode when global leader election or service election is enabled (e.g addresses were announced but were never deleted.
Current state is:
BGP announcement:
Routing table:
Tagging it as draft for now.