Switch from configmap for leader election to lease

samba-in-kubernetes / samba-operator

An operator for a Samba as a service on PVCs in kubernetes

Apache License 2.0

120 stars 24 forks source link

Open phlogistonjohn opened 2 years ago

phlogistonjohn commented 2 years ago

Since we already don't support older k8s versions it probably makes more sense to use the new dedicated resource type.

phlogistonjohn commented 2 years ago

Looks like controller-runtime sets this as the default in version v0.12.0: https://github.com/kubernetes-sigs/controller-runtime/releases/tag/v0.12.0

I'm pretty sure we could override this on the version we currently use. But this could be one reason to consider updating this dependency.