[tempo-distributed] Generic imagePullSecrets helper to remove per-component duplication (#191)

* [tempo-distributed] Replace per-component imagePullSecrets helpers with a single generic helper

Each component had its own _helpers file containing an identical imagePullSecrets
wrapper. This consolidates them into a single `tempo.componentImagePullSecrets`
helper in _helpers.tpl that dynamically resolves the component's values section,
following the same pattern already used by `tempo.imageReference`.

Signed-off-by: Ilia <ilia.lazebnik@gmail.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* [tempo-distributed] Bump chart version to 2.6.5

Signed-off-by: Ilia <ilia.lazebnik@gmail.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Signed-off-by: Ilia <ilia.lazebnik@gmail.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: DrFaust92

charts/tempo-distributed/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: tempo-distributed
 description: Grafana Tempo in MicroService mode
 type: application
-version: 2.6.4
+version: 2.6.5
 # renovate: docker=docker.io/grafana/tempo
 appVersion: 2.10.3
 kubeVersion: "^1.25.0-0"

charts/tempo-distributed/templates/_helpers.tpl

@@ -47,6 +47,28 @@ imagePullSecrets:
 {{- end }}
 {{- end -}}
 
+{{/*
+Generic component imagePullSecrets. Resolves the component's values section
+dynamically, following the same pattern as tempo.imageReference.
+Params:
+  ctx = root context (.)
+  component = component name (e.g., "distributor", "query-frontend")
+  noTempoFallback = optional, when true the tempo image is not used as fallback (for non-tempo images like gateway)
+*/}}
+{{- define "tempo.componentImagePullSecrets" -}}
+{{- $componentSection := include "tempo.componentSectionFromName" . -}}
+{{- if not (hasKey .ctx.Values $componentSection) -}}
+{{- print "Component section " $componentSection " does not exist" | fail -}}
+{{- end -}}
+{{- $component := (index .ctx.Values $componentSection).image | default dict -}}
+{{- $tempo := .ctx.Values.tempo.image -}}
+{{- if .noTempoFallback -}}
+{{- $tempo = dict -}}
+{{- end -}}
+{{- $dict := dict "tempo" $tempo "component" $component "global" .ctx.Values.global.image -}}
+{{- include "tempo.imagePullSecrets" $dict -}}
+{{- end -}}
+
 {{/*
 Create chart name and version as used by the chart label.
 */}}

charts/tempo-distributed/templates/admin-api/_helpers.tpl

@@ -38,7 +38,7 @@ spec:
         {{- with .Values.adminApi.initContainers }}
         {{- toYaml . | nindent 8 }}
         {{- end }}
-      {{- include "tempo.adminApiImagePullSecrets" . | nindent 6 -}}
+      {{- include "tempo.componentImagePullSecrets" (dict "ctx" . "component" "admin-api") | nindent 6 -}}
       {{- with .Values.adminApi.hostAliases }}
       hostAliases:
         {{- toYaml . | nindent 8 }}

charts/tempo-distributed/templates/admin-api/admin-api-dep.yaml

@@ -51,7 +51,7 @@ spec:
         {{- toYaml . | nindent 8 }}
       {{- end }}
       enableServiceLinks: false
-      {{- include "tempo.compactorImagePullSecrets" . | nindent 6 -}}
+      {{- include "tempo.componentImagePullSecrets" (dict "ctx" . "component" "compactor") | nindent 6 -}}
       {{- with .Values.compactor.hostAliases }}
       hostAliases:
         {{- toYaml . | nindent 8 }}

charts/tempo-distributed/templates/compactor/_helpers-compactor.tpl

@@ -51,7 +51,7 @@ spec:
         {{- toYaml . | nindent 8 }}
       {{- end }}
       enableServiceLinks: false
-      {{- include "tempo.distributorImagePullSecrets" . | nindent 6 -}}
+      {{- include "tempo.componentImagePullSecrets" (dict "ctx" . "component" "distributor") | nindent 6 -}}
       {{- with .Values.distributor.hostAliases }}
       hostAliases:
         {{- toYaml . | nindent 8 }}

charts/tempo-distributed/templates/compactor/deployment-compactor.yaml

@@ -52,7 +52,7 @@ spec:
         {{- toYaml . | nindent 8 }}
       {{- end }}
       enableServiceLinks: false
-      {{- include "tempo.enterpriseFederationFrontendImagePullSecrets" . | nindent 6 -}}
+      {{- include "tempo.componentImagePullSecrets" (dict "ctx" . "component" "enterprise-federation-frontend") | nindent 6 -}}
       {{- with .Values.enterpriseFederationFrontend.hostAliases }}
       hostAliases:
         {{- toYaml . | nindent 8 }}