From 3d0caaab19ea1996d49c32d7218646f610718009 Mon Sep 17 00:00:00 2001 From: Andrei Kvapil Date: Wed, 7 May 2025 15:44:20 +0200 Subject: [PATCH] [platform] Automatically exclude dependencies on disabled optional components Signed-off-by: Andrei Kvapil --- packages/core/platform/bundles/paas-full.yaml | 5 ----- packages/core/platform/templates/helmreleases.yaml | 5 ++++- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/packages/core/platform/bundles/paas-full.yaml b/packages/core/platform/bundles/paas-full.yaml index da5a0e8a..95196c8e 100644 --- a/packages/core/platform/bundles/paas-full.yaml +++ b/packages/core/platform/bundles/paas-full.yaml @@ -321,12 +321,7 @@ releases: name: kubeapps-auth-config valuesKey: values.yaml {{- end }} - - {{- if eq $oidcEnabled "true" }} dependsOn: [keycloak-configure] - {{- else }} - dependsOn: [] - {{- end }} - name: kamaji releaseName: kamaji diff --git a/packages/core/platform/templates/helmreleases.yaml b/packages/core/platform/templates/helmreleases.yaml index 1298d74b..64d4084c 100644 --- a/packages/core/platform/templates/helmreleases.yaml +++ b/packages/core/platform/templates/helmreleases.yaml @@ -7,6 +7,9 @@ {{/* collect dependency namespaces from releases */}} {{- range $x := $bundle.releases }} +{{- if or (has $x.name $disabledComponents) (and ($x.optional) (not (has $x.name $enabledComponents))) }} +{{- continue }} +{{- end }} {{- $_ := set $dependencyNamespaces $x.name $x.namespace }} {{- end }} @@ -86,7 +89,7 @@ spec: {{- with $x.dependsOn }} dependsOn: {{- range $dep := . }} - {{- if not (has $dep $disabledComponents) }} + {{- if hasKey $dependencyNamespaces $dep }} - name: {{ $dep }} namespace: {{ index $dependencyNamespaces $dep }} {{- end }}