Skip to content

Commit

Permalink
Readiness probe (#352)
Browse files Browse the repository at this point in the history
* add back port 80 for registry

* add readinessProbe
  • Loading branch information
mxyng committed Sep 28, 2021
1 parent 8efa9b3 commit 6cc8389
Show file tree
Hide file tree
Showing 3 changed files with 22 additions and 3 deletions.
7 changes: 7 additions & 0 deletions helm/charts/engine/templates/deployment.yaml
Expand Up @@ -55,6 +55,13 @@ spec:
readOnly: true
- name: data
mountPath: /var/run/infra/data
readinessProbe:
httpGet:
scheme: HTTPS
path: /healthz
{{- with first .Values.service.ports }}
port: {{ .targetPort }}
{{- end }}
livenessProbe:
httpGet:
scheme: HTTPS
Expand Down
13 changes: 11 additions & 2 deletions helm/charts/registry/templates/deployment.yaml
Expand Up @@ -48,11 +48,20 @@ spec:
mountPath: /var/run/infra/data
- name: config
mountPath: /var/run/infra/config
readinessProbe:
httpGet:
{{- with first .Values.service.ports }}
scheme: {{ .name | upper }}
path: /healthz
port: {{ .targetPort }}
{{- end }}
livenessProbe:
httpGet:
{{- with first .Values.service.ports }}
scheme: {{ .name | upper }}
path: /healthz
scheme: HTTPS
port: 443
port: {{ .targetPort }}
{{- end }}
periodSeconds: 3
affinity:
podAffinity:
Expand Down
5 changes: 4 additions & 1 deletion helm/charts/registry/values.yaml
Expand Up @@ -12,6 +12,9 @@ service:
labels: {}
annotations: {}
ports:
- name: http
port: 80
targetPort: 80
- name: https
port: 443
targetPort: 443
Expand All @@ -20,6 +23,6 @@ ingress:
enabled: false
labels: {}
annotations: {}
servicePort: 443
servicePort: 80
hosts: []
tls: []

0 comments on commit 6cc8389

Please sign in to comment.