-
Notifications
You must be signed in to change notification settings - Fork 45
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
Provide a leader election module #278
Comments
Thanks for raising this @multani. This sounds like a really great feature to add to Looking at the example in |
It also looks like the official Python client uses the previous |
It looks like the official Python client implementation is more like a "demo" rather than a production ready implementation, but I think it's a good idea to offer this mechanism out of the box.
I'm interested to take a look at this and try to sketch up something that would be nice to use and would offer both sync and async APIs. I did a similar experiment on top of Consul's lock/session APIs several months ago, I can come back in a couple of days with some ideas how it could look like using Kubernetes mechanisms instead 👍 |
Sounds great! |
@multani I assume you got busy, it happens! I'm going to drop the assignment and add the "help wanted" label in case anyone else wants to pick it up. |
@jacobtomlinson yes, that's fair 🙇 I still plan to take a look, but it took a bit longer than expected and I've been busy with other things. If someone else wants to move this forward, that would also be awesome! |
Which project are you requesting an enhancement for?
kr8s
What do you need?
Would it be possible to provide an out of the box module that implement leader election directly from kr8s?
The official Kubernetes client for Python provide some kind of implementation directly in the library: https://github.com/kubernetes-client/python/blob/master/kubernetes/base/leaderelection/ and there are well-known pattern to implement leader election using only core primitives of Kubernetes.
If this is something that kr8s would be willing to provide, I would be interested to contribute the feature :)
The text was updated successfully, but these errors were encountered: