Merge pull request #162 from Filipe-Souza/optional-healthz-container

- Modified the chart, now the healthz container/service port is optional
This commit is contained in:
Thijs van Loef
2024-01-28 16:06:26 +01:00
committed by GitHub
5 changed files with 23 additions and 13 deletions

View File

@@ -28,6 +28,7 @@ spec:
{{- end }}
spec:
containers:
{{ if .Values.server.service.healthz.enabled }}
- name: healthz
image: "chussenot/tiny-server:latest"
imagePullPolicy: {{ .Values.server.image.imagePullPolicy }}
@@ -35,6 +36,7 @@ spec:
- name: healthz
containerPort: 80
protocol: TCP
{{ end }}
- name: server
image: "{{ .Values.server.image.name }}:{{ .Values.server.image.tag }}"
imagePullPolicy: {{ .Values.server.image.imagePullPolicy }}

View File

@@ -22,6 +22,12 @@ spec:
selector:
app.kubernetes.io/component: "{{ .Release.Name }}-server"
ports:
{{ if .Values.server.service.healthz.enabled }}
- name: {{ .Values.server.service.healthz.name }}
port: {{ .Values.server.service.healthz.port }}
protocol: {{ .Values.server.service.healthz.protocol }}
targetPort: {{ .Values.server.service.healthz.targetPort }}
{{ end }}
{{- with .Values.server.service.ports }}
{{- toYaml . | nindent 4 }}
{{- end }}