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

Support configuring updateStrategy.rollingUpdate.maxUnavailable in Helm charts #361

Closed
chrisamert opened this issue Oct 23, 2020 · 1 comment · Fixed by #362
Closed

Support configuring updateStrategy.rollingUpdate.maxUnavailable in Helm charts #361

chrisamert opened this issue Oct 23, 2020 · 1 comment · Fixed by #362
Assignees
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@chrisamert
Copy link
Contributor

Describe the solution you'd like
Right now the Helm Chart configure the Daemonset with the RollingUpdate strategy, but don't provide any option so specify the 'maxUnavailble' property, so the default value of '1' is being used. On our 60+ node clusters this is causing the deployment to take 40+ minutes to complete. It would be great if the Helm charts allowed 'maxUnavailable' to be configurable so more than one node can be updated at once.

Anything else you would like to add:
Related issue in the KeyVault Provider repo: Azure/secrets-store-csi-driver-provider-azure#283

Environment:

  • Secrets Store CSI Driver version: (use the image tag):
  • Kubernetes version: (use kubectl version):
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
2 participants