Skip to content

add clickhouse-operator deployment strategy parameters to Helm chart#1789

Merged
sunsingerus merged 2 commits intomasterfrom
improve_helm
Aug 12, 2025
Merged

add clickhouse-operator deployment strategy parameters to Helm chart#1789
sunsingerus merged 2 commits intomasterfrom
improve_helm

Conversation

@Slach
Copy link
Copy Markdown
Collaborator

@Slach Slach commented Aug 12, 2025 •
edited
Loading

For default RollingUpdate deployment.spec.strategy old container will drop after creation of the new container, and we have a small period for two instances of clickhouse-operator could have race conditions

  • All commits in the PR are squashed. More info
  • The PR is made into dedicated next-release branch, not into master branch1. More info
  • The PR is signed. More info

We can separately add this option into dev/ deployment templates.

@Slach Slach requested a review from sunsingerus August 12, 2025 05:02
@sunsingerus sunsingerus merged commit d344625 into master Aug 12, 2025
3 checks passed
@sunsingerus sunsingerus deleted the improve_helm branch August 12, 2025 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sunsingerus sunsingerus sunsingerus approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Slach @sunsingerus