grafana / helm-charts

Apache License 2.0
1.67k stars 2.28k forks source link

[tempo-distributed] Remove duplicated ingester podLabels #3451

Open SimKev2 opened 6 hours ago

SimKev2 commented 6 hours ago

Closes https://github.com/grafana/helm-charts/issues/3237

Ran into an issue when trying to add podLabels into our deploy of tempo.

RUNTIME ERROR: Parsing Helm output: yaml: unmarshal errors:
  line 1752: mapping key "name" already defined at line 1751

Tracking back to the rendering here, it seems like the .Values.ingester.podLabels are duplicated between the statefulset definition and the _helpers.tpl function : https://github.com/grafana/helm-charts/blob/main/charts/tempo-distributed/templates/ingester/_helpers-ingester.tpl#L172-L174