-
Notifications
You must be signed in to change notification settings - Fork 189
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
Allow the 'updateStrategy' of the Daemonset to be configured in Helm #285
Allow the 'updateStrategy' of the Daemonset to be configured in Helm #285
Conversation
/azp run pr-e2e-azure |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@chrisamert These changes need to be made in the manifest_staging
dir where we currently host the staging charts. All the chart changes are then tagged with the release. Could you make the changes only in manifest_staging/charts instead
Ahh I missed that. I'll get these changes moved over |
This PR enables the 'updateStrategy' of the Daemonset to be configured by Helm parameters. The default values are the same as the current defaults; RollingUpdate strategy with maxUnavailable set to 1, so behavior will change only if the user specifies a custom value.
a51a325
to
422edb4
Compare
Adding the default values from docs here for future reference-
Ref docs: |
/azp run pr-e2e-azure |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Reason for Change:
This PR enables the 'updateStrategy' of the Daemonset to be configured by Helm parameters. The default values are the same as the current defaults; RollingUpdate strategy with maxUnavailable set to 1, so behavior will change only if the user specifies a custom value.
Requirements
Issue Fixed:
Fixes #283 283
Please answer the following questions with yes/no:
Special Notes for Reviewers: