From 52d749d46a6131a406ffcdb2921a8827007030fd Mon Sep 17 00:00:00 2001 From: Denis Yudin Date: Thu, 4 Sep 2025 12:13:22 +0100 Subject: [PATCH] fix: use mergeOverwrite to properly override ConfigMap values Fixes an issue where boolean values from bundle files were not being properly overridden by values- ConfigMap entries. The Helm merge function has a bug when merging boolean values where true from the first dict doesn't get overwritten by false from the second dict. Using mergeOverwrite ensures ConfigMap values take precedence over bundle values as intended. Example: - Bundle: autoDirectNodeRoutes: true - ConfigMap values-cilium: autoDirectNodeRoutes: false - Before: result was true (incorrect) - After: result is false (correct) This fix ensures that users can properly override any component configuration using the values- pattern in the cozystack ConfigMap. Signed-off-by: Denis Yudin --- packages/core/platform/templates/helmreleases.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/core/platform/templates/helmreleases.yaml b/packages/core/platform/templates/helmreleases.yaml index 269f3750..6ed61ed8 100644 --- a/packages/core/platform/templates/helmreleases.yaml +++ b/packages/core/platform/templates/helmreleases.yaml @@ -77,7 +77,7 @@ spec: {{- $values = merge . $values }} {{- end }} {{- with index $cozyConfig.data (printf "values-%s" $x.name) }} - {{- $values = merge (fromYaml .) $values }} + {{- $values = mergeOverwrite $values (fromYaml .) }} {{- end }} {{- with $values }} values: