Skip to content

Commit 3f36fbc

Browse files
sringeltincho-burgos
authored andcommitted
[tempo-distributed] bugfix: ingester zone specific statefulset and pod annotations
Signed-off-by: Sean Ringel <[email protected]>
1 parent bd2783a commit 3f36fbc

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
{{- end }}
55
{{- define "ingester.zoneAwareReplicationMap" -}}
66
{{- $zonesMap := (dict) -}}
7-
{{- $defaultZone := (dict "affinity" .ctx.Values.ingester.affinity "nodeSelector" .ctx.Values.ingester.nodeSelector "replicas" .ctx.Values.ingester.replicas "storageClass" .ctx.Values.ingester.storageClass) -}}
7+
{{- $defaultZone := (dict "affinity" .ctx.Values.ingester.affinity "annotations" (default (dict)) "nodeSelector" .ctx.Values.ingester.nodeSelector "podAnnotations" (default (dict)) "nodeSelector" "replicas" .ctx.Values.ingester.replicas "storageClass" .ctx.Values.ingester.storageClass) -}}
88
{{- if .ctx.Values.ingester.zoneAwareReplication.enabled -}}
99
{{- $numberOfZones := len .ctx.Values.ingester.zoneAwareReplication.zones -}}
1010
{{- if lt $numberOfZones 3 -}}
@@ -16,7 +16,7 @@
1616
{{- $extraAffinity := $rolloutZone.extraAffinity | default (dict) -}}
1717
{{- $zoneAntiAffinity := include "ingester.zoneAntiAffinity" (dict "rolloutZoneName" $rolloutZone.name "topologyKey" $.ctx.Values.ingester.zoneAwareReplication.topologyKey) | fromYaml -}}
1818
{{- $mergedAffinity := mergeOverwrite $extraAffinity $zoneAntiAffinity -}}
19-
{{- $_ := set $zonesMap $rolloutZone.name (dict "affinity" $mergedAffinity "nodeSelector" ($rolloutZone.nodeSelector | default (dict)) "replicas" $replicaPerZone "storageClass" $rolloutZone.storageClass) -}}
19+
{{- $_ := set $zonesMap $rolloutZone.name (dict "affinity" $mergedAffinity "annotations" ($rolloutZone.annotations | default (dict)) "nodeSelector" ($rolloutZone.nodeSelector | default (dict)) "podAnnotations" ($rolloutZone.podAnnotations | default (dict)) "replicas" $replicaPerZone "storageClass" $rolloutZone.storageClass) -}}
2020
{{- end -}}
2121
{{- else -}}
2222
{{- $_ := set $zonesMap "" $defaultZone -}}

charts/tempo-distributed/templates/ingester/statefulset-ingester.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ metadata:
1515
{{- if .Values.ingester.zoneAwareReplication.enabled }}
1616
annotations:
1717
{{- include "ingester.Annotations" $dict | nindent 4}}
18-
{{- with $rolloutZone.annotations }}:
18+
{{- with $rolloutZone.annotations }}
1919
{{- toYaml . | nindent 4 }}
2020
{{- end }}
2121
{{- else }}
@@ -54,7 +54,7 @@ spec:
5454
{{- with .Values.ingester.podAnnotations }}
5555
{{- toYaml . | nindent 8 }}
5656
{{- end }}
57-
{{- with $rolloutZone.podAnnotations }}:
57+
{{- with $rolloutZone.podAnnotations }}
5858
{{- toYaml . | nindent 8 }}
5959
{{- end }}
6060
spec:

0 commit comments

Comments
 (0)