Skip to content

feat(clickhouse): make operator configs customizable - #903

Open
abhinavmathur-atlan wants to merge 3 commits into
SigNoz:mainfrom
abhinavmathur-atlan:feat/575-customize-operator-configs
Open

feat(clickhouse): make operator configs customizable#903
abhinavmathur-atlan wants to merge 3 commits into
SigNoz:mainfrom
abhinavmathur-atlan:feat/575-customize-operator-configs

Conversation

@abhinavmathur-atlan

@abhinavmathur-atlan abhinavmathur-atlan commented Aug 7, 2026

Copy link
Copy Markdown

Summary

  • allow replacing all five ClickHouse Operator configuration file maps through values
  • preserve byte-identical default rendering when overrides are unset
  • render each custom file before YAML serialization and reject non-string values clearly
  • handle clickhouseOperator.configs: null safely
  • add focused Helm unittest coverage

Testing

  • helm lint charts/clickhouse
  • full default render compared byte-for-byte with origin/main
  • full custom render with all five ConfigMaps parsed and checked for exact round-trip values
  • null-config render succeeds
  • structured/non-string input fails with the expected validation error
  • independent adversarial Claude review: APPROVE

Authored with Codex and independently adversarially reviewed with Claude.

Fixes #575

@abhinavmathur-atlan
abhinavmathur-atlan requested a review from a team as a code owner August 7, 2026 03:12
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.

Enable more customization in the clickhouse operator

1 participant