Fixing Tempo Registry Colcase Order

Problem: Component-specific registry overrides weren't working despite being documented as overriding global registry.

  Root Cause: Template helper function had incorrect priority order in coalesce:
  `{{- $registry := coalesce .global.registry .component.registry .tempo.registry -}}`

  Fix: Changed priority to put component registry first:
  `{{- $registry := coalesce .component.registry .tempo.registry .global.registry -}}``

  Result:
  - Component registry now properly overrides global registry as documented
  - Fully backwards compatible - when component registry is null, falls back to global
  - Enables clean configuration like memcached.image.registry: public.ecr.aws

Signed-off-by: portswigger-amason <[email protected]>

Author: portswigger-amason

charts/tempo-distributed/templates/_helpers.tpl

@@ -28,7 +28,7 @@ If release name contains chart name it will be used as a full name.
 Docker image selector for Tempo. Hierachy based on global, component, and tempo values.
 */}}
 {{- define "tempo.tempoImage" -}}
-{{- $registry := coalesce .global.registry .component.registry .tempo.registry -}}
+{{- $registry := coalesce .component.registry .tempo.registry .global.registry -}}
 {{- $repository := coalesce .component.repository .tempo.repository -}}
 {{- $tag := coalesce .component.tag .tempo.tag .defaultVersion | toString -}}
 {{- printf "%s/%s:%s" $registry $repository $tag -}}