Skip to content

Conversation

@Slach
Copy link
Collaborator

@Slach Slach commented Sep 15, 2025

fix #1814

  • 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

@Slach Slach requested a review from sunsingerus September 15, 2025 17:18
Slach and others added 4 commits September 16, 2025 20:05
Co-authored-by: aider (openrouter/qwen/qwen3-coder) <[email protected]>
Co-authored-by: aider (openrouter/qwen/qwen3-coder) <[email protected]>
Co-authored-by: aider (openrouter/qwen/qwen3-coder) <[email protected]>
…helm_chart.sh, remove duplicated strategy from Deployment-clickhouse-operator.yaml
@Slach Slach requested a review from alex-zaitsev September 16, 2025 16:00
# if [[ "0" == $(helm plugin list | grep -c schema) ]]; then
# helm plugin install https://github.com/losisin/helm-values-schema-json.git
# fi
# helm schema --use-helm-docs -f "${values_yaml}" --output "${chart_path}/values.schema.json"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why this code that generates values.schema.json is commented now? Schema may need to be updated from time to time, does it not?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately, it generates a schema that I need to manually change later. =(

@sunsingerus sunsingerus merged commit 63efaea into 0.25.4 Sep 19, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants