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

enable leaderelection for bgp in CP mode #578

Merged

Conversation

marc-cerebras
Copy link
Contributor

Hi kube-vip, in order to implement a BGP based VIP with leader election, I had to make this tweak.

I have tested this on a test cluster in control plane however a notable deficiency in my testing is that I have not tried this change in services mode. It does not look like there is an automated test for BGP so I could use some advice on how to proceed with this change- Thank you!

Signed-off-by: marc-cerebras <marc.spehlmann@cerebras.net>
@marc-cerebras marc-cerebras force-pushed the marc-cerebras/bgp-cp-leaderelect branch from 26074c4 to 2dcd9e5 Compare June 19, 2023 18:25
@thebsdbox
Copy link
Collaborator

This looks ok, I'm intrigued about the use case. With BGP a leader election isn't required as the ToR switches will effectively just choose a down stream route. (either way happy to merge)!

Apologies for the delay I've been on PTO 🏖️

@marc-cerebras
Copy link
Contributor Author

I'm intrigued about the use case

Sure - one simple reason is for predictability of which node is receiving traffic in situations where there's multiple ToR switches where our controlplane is on multiple racks, we want to have this configuration option.

The other reason is for a non-controlplane use-case where we're launching kv in control-plane mode - we don't want clients to get connection resets in the case that one node hosting a kv follower goes down which triggers routers to reconfigure, resetting connections.

(either way happy to merge)!

👍

@marc-cerebras
Copy link
Contributor Author

@yastij any objections or suggestions prior to merge?

@thebsdbox
Copy link
Collaborator

Looks good to me!

@thebsdbox thebsdbox merged commit ec80533 into kube-vip:main Jul 18, 2023
4 checks passed
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.

None yet

2 participants