helm-charts icon indicating copy to clipboard operation
helm-charts copied to clipboard

Support for restart policy in sftpgo Helm Chart

Open agmimidi opened this issue 2 years ago • 4 comments

Hi,

We are seeing an issue where when we upgrade our chart the new pod gets stuck in Container Creating state.

This happens because the old pod is still there and holds access to the persistent volume claim. The only way around this at the moment is to force delete the old pod. Adding support for pod restart policy or allowing deploying as a state-full set should solve the issue.

Cheers and thanks for the chart :) Angelos

agmimidi avatar Mar 21 '22 15:03 agmimidi

@agmimidi which chart are you talking about?

sagikazarmark avatar Mar 21 '22 17:03 sagikazarmark

https://github.com/sagikazarmark/helm-charts/tree/master/charts/sftpgo

Tis on the issue title 😁

agmimidi avatar Mar 21 '22 19:03 agmimidi

@sagikazarmark Did you have a chance to look into this?

Cheers, Angelos

agmimidi avatar Mar 29 '22 12:03 agmimidi

Sorry, I have not.

I'm a bit confused though: Isn't deployment strategy that you want to change: https://kubernetes.io/docs/concepts/workloads/controllers/deployment/#strategy

sagikazarmark avatar Mar 29 '22 14:03 sagikazarmark