Skip to content

Commit c833f1a

Browse files
authored
fix(helm): Avoid nil pointer for metrics.enabled inside podAnnotations (#6251)
* Fix helm podAnnotations metrics context Signed-off-by: antznette1 <ochiezeanthonette@gmail.com> * Fix helm metrics annotation with empty podAnnotations Signed-off-by: antznette1 <ochiezeanthonette@gmail.com> --------- Signed-off-by: antznette1 <ochiezeanthonette@gmail.com>
1 parent c8fd816 commit c833f1a

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

infra/charts/feast-feature-server/templates/deployment.yaml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,12 @@ spec:
1111
{{- include "feast-feature-server.selectorLabels" . | nindent 6 }}
1212
template:
1313
metadata:
14-
{{- with .Values.podAnnotations }}
14+
{{- if or .Values.podAnnotations .Values.metrics.enabled }}
1515
annotations:
16+
{{- with .Values.podAnnotations }}
1617
{{- toYaml . | nindent 8 }}
17-
{{- if .Values.metrics.enabled }}
18+
{{- end }}
19+
{{- if $.Values.metrics.enabled }}
1820
instrumentation.opentelemetry.io/inject-python: "true"
1921
{{- end }}
2022
{{- end }}

0 commit comments

Comments
 (0)