fix(helm): wire watchNamespaces value into operator ConfigMap

[jtomaszon]

Problem

Closes #1919.

configs.files.config.yaml.watch.namespaces.include is hardcoded to [] in values.yaml and the ConfigMap template renders it directly — there is no way to override it via a top-level Helm value. Users who need the operator to watch multiple namespaces must manually patch the ConfigMap after install, which is fragile and breaks on upgrades.

Solution

This PR introduces:

1. A top-level watchNamespaces value in values.yaml (empty by default — preserves existing behavior):

# watchNamespaces -- namespaces where the operator watches for ClickHouseInstallation resources.
# If empty, the operator watches only its own namespace (or all namespaces when running in kube-system).
# Use [".*"] to watch all namespaces.
watchNamespaces: []

2. A new altinity-clickhouse-operator.configmap-files helper in _helpers.tpl that deep-copies configs.files, patches watch.namespaces.include in-place, and delegates to the existing configmap-data helper for rendering.

3. The ConfigMap template updated to call the new helper instead of configmap-data directly.

Usage

# values.yaml
watchNamespaces:
  - clickhouse-system
  - my-app-namespace

Or via --set:

helm install clickhouse-operator altinity/altinity-clickhouse-operator \
  --set "watchNamespaces={clickhouse-system,my-app-namespace}"

Behavior

• watchNamespaces: [] (default) — no change, operator watches its own namespace
• watchNamespaces: [".*"] — watch all namespaces
• Any non-empty list — operator watches exactly those namespaces

Testing

# Default — include stays []
helm template test deploy/helm/clickhouse-operator \
  --show-only templates/generated/ConfigMap-etc-clickhouse-operator-files.yaml | grep -A5 'watch:'

# With namespaces set
helm template test deploy/helm/clickhouse-operator \
  --set 'watchNamespaces={clickhouse-system,everest-dev}' \
  --show-only templates/generated/ConfigMap-etc-clickhouse-operator-files.yaml | grep -A8 'watch:'

[sunsingerus]

PR includes modified files that are auto-generated and no modifications to the generator itself

[jtomaszon]

Thanks for the review — you're right. I've reworked it so the change comes from the generator, not the generated file.

dev/generate_helm_chart.sh now special-cases the etc-clickhouse-operator-files ConfigMap in update_configmap_resource(), emitting the configmap-files helper (which wires watchNamespaces into watch.namespaces.include) instead of the generic configmap-data. Re-running generate_helm_chart.sh now reproduces the committed template with no manual edits. The README values table was regenerated by helm-docs.

Verified: default (watchNamespaces: []) leaves include empty; --set 'watchNamespaces={ns1,ns2}' renders them under watch.namespaces.include.