openebs / lvm-localpv

Dynamically provision Stateful Persistent Node-Local Volumes & Filesystems for Kubernetes that is integrated with a backend LVM2 data storage stack.
Apache License 2.0
235 stars 92 forks source link

fix(charts): Change lvm-controller's Kind from StatefulSet to Deployment (#196) #258

Closed evgkrsk closed 4 months ago

evgkrsk commented 10 months ago

Pull Request template

Why is this PR required? What issue does it fix?:

This PR address #196 issue and tries to avoid lvm-controller's disruption if underlaying node(s) crashed. Deployment's controller will restart failed pod(s) on healthy node, unlike StatefulSet's controller.

What this PR does?:

It change lvm-controller's Kind from StatefuleSet to Deployment.

Does this PR require any upgrade changes?:

No, as far as I know.

If the changes in this PR are manually verified, list down the scenarios covered::

Checklist:

Abhinandan-Purkait commented 4 months ago

Closing since it's cherry-picked in #284