CRD in Operator 0.25.5 have invalid value for watched namespaces

[morkalfon]

After upgrading from ClickHouse Operator 0.25.3 to 0.25.5, our previous ClickHouseOperatorConfiguration manifests—valid and working on 0.25.3—are now rejected due to a breaking change in the CRD schema for the watch.namespaces field.

This is a YAML snippet of the relevant configuration (which worked in v0.25.3):

apiVersion: clickhouse.altinity.com/v1
kind: ClickHouseOperatorConfiguration
# ...
spec:
  # ...
  watch:
    namespaces:
      - clickhouse

• In 0.25.3, the CRD defines spec.watch.namespaces as an array of strings.
• In 0.25.5, this changed to an object (type: object), breaking compatibility with configurations that previously used the old array format or documentation that expected one behavior.

CRD change that caused breakage:

• 0.25.3 CRD lines:

namespaces:
  type: array
  description: "List of namespaces where clickhouse-operator watches for events."
  items:
    type: string

• 0.25.5 CRD lines:

namespaces:
  type: object
  description: "List of namespaces where clickhouse-operator watches for events."
  x-kubernetes-preserve-unknown-fields: true

[Slach]

accodring to https://github.com/Altinity/clickhouse-operator/blame/50e8d0490678185f290336acbdfbf9534d60367d/deploy/operator/parts/crd.yaml#L7160

it looks like was bugfix, and before just not works

how did you test array format in 0.25.3?
did you try to add custom resources in other namespaces then clickhouse?
which namespace do you use to install clickhouse-operator?

[morkalfon]

@Slach See example in the above message.

[morkalfon]

@Slach Let me know if you need more information from me. We deploy the operator to the clickhouse namespace and it failed to do so with this new CRD. This is the error we were getting when applied: Invalid value: "array": spec.watch.namespaces in body must be of type object: "array"

[Slach]

@morkalfon i read your example

i mean following:
your CRD in "old" format looks like just didn't work
i mean it loaded without error
but not resticted anything

if clickouse-operator deployment and kind: ClickHouseInstallation custom resources in the same clickhouse namespace, then clickouse-operator will watch CHI in the same namespace by default

so, you just need change your kind: ClickHouseOperatorConfiguration to new format to apply changes

is anything else what do you need to change with your kind: ClickHouseOperatorConfiguration?

[morkalfon]

@Slach I'm not sure I'm following you. We want to supply a list of namespaces to be watched by the operator, so how it can be done in the new CRD:

clickhouse-operator/deploy/operatorhub/0.25.5/clickhouseoperatorconfigurations.clickhouse.altinity.com.crd.yaml

Lines 51 to 58 in 50e8d04

	watch:
	type: object
	description: "Parameters for watch kubernetes resources which used by clickhouse-operator deployment"
	properties:
	namespaces:
	type: object
	description: "List of namespaces where clickhouse-operator watches for events."
	x-kubernetes-preserve-unknown-fields: true

what should be the new format?
In regards to what else do we need to change in kind: ClickHouseOperatorConfiguration we do change several things there. It's not just that.

[sunsingerus]

what should be the new format?

clickhouse-operator/docs/chi-examples/70-chop-config.yaml

Lines 11 to 19 in 50e8d04

	watch:
	# List of namespaces where clickhouse-operator watches for events.
	# Concurrently running operators should watch on different namespaces.
	# IMPORTANT
	# Regexp is applicable.
	namespaces:
	include: []
	exclude: []