Skip to content

Commit

Permalink
[K8S][HELM] Address some PR comments
Browse files Browse the repository at this point in the history
  • Loading branch information
dnskr committed Jan 15, 2023
1 parent 36b5fc6 commit 8ddef6a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion charts/kyuubi/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
*/}}

{{/*
A comma separated string of enabled frontend protocols, e.g. "MYSQL,REST,THRIFT_BINARY".
A comma separated string of enabled frontend protocols, e.g. "REST,THRIFT_BINARY".
For details, see 'kyuubi.frontend.protocols': https://kyuubi.readthedocs.io/en/master/deployment/settings.html#frontend
*/}}
{{- define "kyuubi.frontend.protocols" -}}
Expand Down
4 changes: 2 additions & 2 deletions charts/kyuubi/templates/kyuubi-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ spec:
{{- if .Values.probe.liveness.enabled }}
livenessProbe:
exec:
command: ["/bin/sh", "-c", "bin/kyuubi status"]
command: ["/bin/bash", "-c", "bin/kyuubi status"]
initialDelaySeconds: {{ .Values.probe.liveness.initialDelaySeconds }}
periodSeconds: {{ .Values.probe.liveness.periodSeconds }}
timeoutSeconds: {{ .Values.probe.liveness.timeoutSeconds }}
Expand All @@ -76,7 +76,7 @@ spec:
{{- if .Values.probe.readiness.enabled }}
readinessProbe:
exec:
command: [ "/bin/sh", "-c", "$KYUUBI_HOME/bin/kyuubi status" ]
command: ["/bin/bash", "-c", "$KYUUBI_HOME/bin/kyuubi status"]
initialDelaySeconds: {{ .Values.probe.readiness.initialDelaySeconds }}
periodSeconds: {{ .Values.probe.readiness.periodSeconds }}
timeoutSeconds: {{ .Values.probe.readiness.timeoutSeconds }}
Expand Down

0 comments on commit 8ddef6a

Please sign in to comment.