From e0af18c12a70c7a09b3ff54d27454e60944b8258 Mon Sep 17 00:00:00 2001 From: stxkxs <139715017+stxkxs@users.noreply.github.com> Date: Sat, 8 Aug 2026 13:48:52 -0700 Subject: [PATCH] chore(deps): otel-collector 0.169.0, with the image following the chart MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Three chart pins move 0.166.0 -> 0.169.0, and the three hand-set image tags come out. The tags are the point. Chart 0.166.0 declares appVersion 0.157.0 and all three values files pinned the image at 0.157.0, so the two agreed — but only by hand, and only for as long as nobody moved one. Chart 0.169.0 declares appVersion 0.158.0, so bumping the chart alone would have opened a divergence where none existed, in the direction that misleads: the chart ahead of the binary it names. Nothing would have reported it. Renovate reaches the chart pin through the argocd manager and does not read these values files at all — eks-gitops sets enabledManagers to argocd, gomod, custom.regex and github-actions, with no helm-values — so it moves the chart and cannot move the image. That is how the two spent a stretch a full release apart before. The chart renders `.Values.image.tag | default .Chart.AppVersion`, so deleting the tag makes the image a function of the pin rather than a second fact maintained beside it. Every future chart bump now carries its image with no second act. Two things checked rather than assumed. The rendered Deployment resolves to otel/opentelemetry-collector-contrib:0.158.0, so the default is reached and the values files are not overriding it somewhere else. And the contrib distro genuinely publishes the version numbers the appVersion tracks — that appVersion follows the core collector, and contrib is a different repository — confirmed against the registry for both 0.157.0 and 0.158.0. task validate passes: 109 addon×env combinations, 0 failed, and no 0.157.0 remains anywhere under addons/ or applicationsets/. kx mirrors this chart and follows in a paired change. --- addons/observability/otel-agent/values.yaml | 9 ++++++++- addons/observability/otel-gateway-floor/values.yaml | 9 ++++++++- addons/observability/otel-gateway/values.yaml | 9 ++++++++- applicationsets/addons-otel-agent.yaml | 2 +- applicationsets/addons-otel-gateway-floor.yaml | 2 +- applicationsets/addons-otel-gateway.yaml | 2 +- 6 files changed, 27 insertions(+), 6 deletions(-) diff --git a/addons/observability/otel-agent/values.yaml b/addons/observability/otel-agent/values.yaml index fe0d6f1..02b439b 100644 --- a/addons/observability/otel-agent/values.yaml +++ b/addons/observability/otel-agent/values.yaml @@ -18,7 +18,14 @@ mode: daemonset # artifact. image.repository is mandatory — chart >=0.110 ships no default. image: repository: otel/opentelemetry-collector-contrib - tag: "0.157.0" + # No `tag:` deliberately. The chart renders + # `.Values.image.tag | default .Chart.AppVersion`, so leaving it unset makes the + # image follow the chart pin instead of being maintained beside it. A hand-set + # tag here has to be moved by a second act nothing performs or checks — Renovate + # sees the chart (argocd manager) and not this file — which is how the two spent + # a stretch a release apart. The contrib distro publishes the same version + # numbers as the core collector the appVersion tracks, verified against the + # registry for the version this pin resolves to. # Stable object names (DaemonSet/ConfigMap/ClusterRole) independent of the ArgoCD # release name, so the gateway address and RBAC bindings don't shift on a rename. diff --git a/addons/observability/otel-gateway-floor/values.yaml b/addons/observability/otel-gateway-floor/values.yaml index 1bf7138..fdf170c 100644 --- a/addons/observability/otel-gateway-floor/values.yaml +++ b/addons/observability/otel-gateway-floor/values.yaml @@ -17,7 +17,14 @@ mode: deployment image: repository: otel/opentelemetry-collector-contrib - tag: '0.157.0' + # No `tag:` deliberately. The chart renders + # `.Values.image.tag | default .Chart.AppVersion`, so leaving it unset makes the + # image follow the chart pin instead of being maintained beside it. A hand-set + # tag here has to be moved by a second act nothing performs or checks — Renovate + # sees the chart (argocd manager) and not this file — which is how the two spent + # a stretch a release apart. The contrib distro publishes the same version + # numbers as the core collector the appVersion tracks, verified against the + # registry for the version this pin resolves to. # Identical object names to the full tier, deliberately: the node agent forwards # to otel-gateway.monitoring.svc.cluster.local:4317 and the telemetry alias diff --git a/addons/observability/otel-gateway/values.yaml b/addons/observability/otel-gateway/values.yaml index 19e8049..52e5b29 100644 --- a/addons/observability/otel-gateway/values.yaml +++ b/addons/observability/otel-gateway/values.yaml @@ -15,7 +15,14 @@ mode: deployment image: repository: otel/opentelemetry-collector-contrib - tag: "0.157.0" + # No `tag:` deliberately. The chart renders + # `.Values.image.tag | default .Chart.AppVersion`, so leaving it unset makes the + # image follow the chart pin instead of being maintained beside it. A hand-set + # tag here has to be moved by a second act nothing performs or checks — Renovate + # sees the chart (argocd manager) and not this file — which is how the two spent + # a stretch a release apart. The contrib distro publishes the same version + # numbers as the core collector the appVersion tracks, verified against the + # registry for the version this pin resolves to. # Stable object names — the Service the agents forward to (otel-gateway) and the # SA the AMP Pod Identity association binds must not shift on an ArgoCD rename. diff --git a/applicationsets/addons-otel-agent.yaml b/applicationsets/addons-otel-agent.yaml index f840f5c..f46fac4 100644 --- a/applicationsets/addons-otel-agent.yaml +++ b/applicationsets/addons-otel-agent.yaml @@ -40,7 +40,7 @@ spec: sources: - repoURL: https://open-telemetry.github.io/opentelemetry-helm-charts chart: opentelemetry-collector - targetRevision: 0.166.0 + targetRevision: 0.169.0 helm: releaseName: otel-agent valueFiles: diff --git a/applicationsets/addons-otel-gateway-floor.yaml b/applicationsets/addons-otel-gateway-floor.yaml index a036cca..4c9b302 100644 --- a/applicationsets/addons-otel-gateway-floor.yaml +++ b/applicationsets/addons-otel-gateway-floor.yaml @@ -41,7 +41,7 @@ spec: sources: - repoURL: https://open-telemetry.github.io/opentelemetry-helm-charts chart: opentelemetry-collector - targetRevision: 0.166.0 + targetRevision: 0.169.0 helm: releaseName: otel-gateway # extraEnvs is a list, so Helm replaces it wholesale — valuesObject diff --git a/applicationsets/addons-otel-gateway.yaml b/applicationsets/addons-otel-gateway.yaml index 0713302..3f922f6 100644 --- a/applicationsets/addons-otel-gateway.yaml +++ b/applicationsets/addons-otel-gateway.yaml @@ -38,7 +38,7 @@ spec: sources: - repoURL: https://open-telemetry.github.io/opentelemetry-helm-charts chart: opentelemetry-collector - targetRevision: 0.166.0 + targetRevision: 0.169.0 helm: releaseName: otel-gateway # extraEnvs is a list, so Helm replaces it wholesale — valuesObject