From 80411be4e903ee0d3ad45a26d51317df4b392054 Mon Sep 17 00:00:00 2001 From: stxkxs Date: Sat, 30 May 2026 20:26:03 -0700 Subject: [PATCH] fix(applicationsets): stop injecting clusterName/vpcId into bootstrap addons MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The addons-bootstrap ApplicationSet passed clusterName + vpcId as Helm --set parameters to every bootstrap addon, but none of them (cert-manager, external-secrets, prometheus-operator-crds, metrics-server, reloader) consume those values — they're for aws-load-balancer-controller in the networking set. cert-manager's chart has a strict values schema (additionalProperties: false), so the unknown parameters made it fail to render with a ComparisonError, leaving it permanently OutOfSync/unsynced. Drop the parameters from the bootstrap template; the addons that genuinely need clusterName/vpcId keep them in their own ApplicationSets. --- applicationsets/addons-bootstrap.yaml | 5 ----- 1 file changed, 5 deletions(-) diff --git a/applicationsets/addons-bootstrap.yaml b/applicationsets/addons-bootstrap.yaml index f698d3b..ab364f0 100644 --- a/applicationsets/addons-bootstrap.yaml +++ b/applicationsets/addons-bootstrap.yaml @@ -65,11 +65,6 @@ spec: targetRevision: '{{ .chartVersion }}' helm: releaseName: '{{ .appName }}' - parameters: - - name: clusterName - value: '{{ index .metadata.labels "cluster_name" }}' - - name: vpcId - value: '{{ index .metadata.labels "vpc_id" }}' valueFiles: - $values/{{ .path }}/values.yaml - $values/{{ .path }}/values-{{ index .metadata.labels "environment" }}.yaml