Skip to content

Commit 7c9873b

Browse files
fix(registry): change readiness and liveness probe to httpGet /v2 (#720)
#### What this PR does / why we need it There are lot of error logging and noise in the registry deployment due to TLS errors on the probes - changing the Liveness and Readiness probes `httpGet` on `/v2` path to avoid TLS handshake errors Additionally there are noises from OTEL metrics (enabled by default in registry) but they can be overridden with env via values ```yaml - name: OTEL_TRACES_EXPORTER value: none ``` This is a very small fix and may not fit into v2 controllers but until then it would be nice to get this fix in. Signed-off-by: abhijith-darshan <[email protected]>
1 parent ca46af8 commit 7c9873b

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

deploy/templates/deployment_registry.yaml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,16 +34,20 @@ spec:
3434
{{- toYaml .Values.registry.tls.volumeMounts | nindent 12 }}
3535
{{- end}}
3636
readinessProbe:
37-
tcpSocket:
37+
httpGet:
38+
path: /v2/
3839
port: 5000
40+
scheme: HTTPS
3941
initialDelaySeconds: 2
4042
timeoutSeconds: 1
4143
periodSeconds: 5
4244
successThreshold: 1
4345
failureThreshold: 3
4446
livenessProbe:
45-
tcpSocket:
47+
httpGet:
48+
path: /v2/
4649
port: 5000
50+
scheme: HTTPS
4751
initialDelaySeconds: 15
4852
timeoutSeconds: 1
4953
periodSeconds: 20

0 commit comments

Comments
 (0)