Merge pull request #117 from nicolajv/main

fix incorrect fields in probes and add support for service annotations

Author: jessebot

charts/home-assistant/templates/deployment.yaml

@@ -89,11 +89,11 @@ spec:
           {{- end }}{{/* end check for env vars for home asisstant container */}}
           {{- if .Values.livenessProbe.enabled }}
           livenessProbe:
-            {{- toYaml .Values.livenessProbe | nindent 12 }}
+            {{- toYaml (omit .Values.livenessProbe "enabled") | nindent 12 }}
           {{- end }}
           {{- if .Values.readinessProbe.enabled }}
           readinessProbe:
-            {{- toYaml .Values.readinessProbe | nindent 12 }}
+            {{- toYaml (omit .Values.readinessProbe "enabled") | nindent 12 }}
           {{- end }}
           {{- with .Values.resources }}
           resources:

charts/home-assistant/templates/service.yaml

@@ -4,6 +4,10 @@ metadata:
   name: {{ include "home-assistant.fullname" . }}
   labels:
     {{- include "home-assistant.labels" . | nindent 4 }}
+  {{- with .Values.service.annotations }}
+  annotations:
+    {{- toYaml . | nindent 4 }}
+  {{- end }}
 spec:
   type: {{ .Values.service.type }}
   ports:

charts/home-assistant/values.yaml

@@ -45,6 +45,7 @@ securityContext: {}
   # runAsUser: 1000
 
 service:
+  annotations: {}
   type: ClusterIP
   # -- default port to expose
   port: 80