From 0afdff2866edff80943f1821c328bba9857d8e4f Mon Sep 17 00:00:00 2001 From: Lars Francke Date: Fri, 7 Aug 2026 10:15:35 +0200 Subject: [PATCH] fix: Remove the hardcoded image repository from values.yaml commons-operator was the only operator setting image.repository in its own values.yaml. The other 13 non-CSI operators leave it to the registry overlay in deploy/helm/commons-operator/values/, which publish-helm-chart merges on top at package time. The value was also wrong in shape. operator.image renders "{repository}/{Chart.Name}:{tag}", so repository is the namespace and the chart name is appended. The hardcoded value already contained the name, so had it ever taken effect it would have produced .../sdp/commons-operator/commons-operator. No behaviour change for published charts: the overlay wins the merge either way. --- deploy/helm/commons-operator/values.yaml | 1 - 1 file changed, 1 deletion(-) diff --git a/deploy/helm/commons-operator/values.yaml b/deploy/helm/commons-operator/values.yaml index b8de381c..d43996dc 100644 --- a/deploy/helm/commons-operator/values.yaml +++ b/deploy/helm/commons-operator/values.yaml @@ -1,7 +1,6 @@ # Default values for commons-operator. --- image: - repository: oci.stackable.tech/sdp/commons-operator pullPolicy: IfNotPresent pullSecrets: []