Skip to content

Commit

Permalink
[KYUUBI #4043] [K8S][HELM] Add volumes and volumeMounts support to he…
Browse files Browse the repository at this point in the history
…lm chart

### _Why are the changes needed?_
The changes allow to add volumes to Kyuubi server pod and volumeMounts to  Kyuubi server container.

### _How was this patch tested?_
- [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

- [ ] Add screenshots for manual tests if appropriate

- [ ] [Run test](https://kyuubi.apache.org/docs/latest/develop_tools/testing.html#running-tests) locally before make a pull request

Closes #4043 from dnskr/add_volumes_and_volumeMounts.

Closes #4043

a332b92 [dnskr] [K8S][HELM] Add volumes and volumeMounts support to helm chart

Authored-by: dnskr <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
  • Loading branch information
dnskr authored and pan3793 committed Dec 30, 2022
1 parent 0dfe089 commit e9b713f
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 0 deletions.
6 changes: 6 additions & 0 deletions charts/kyuubi/templates/kyuubi-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -83,10 +83,16 @@ spec:
volumeMounts:
- name: conf
mountPath: {{ .Values.server.confDir }}
{{- with .Values.volumeMounts }}
{{- tpl (toYaml .) $ | nindent 12 }}
{{- end }}
volumes:
- name: conf
configMap:
name: {{ .Release.Name }}
{{- with .Values.volumes }}
{{- tpl (toYaml .) $ | nindent 8 }}
{{- end }}
{{- with .Values.nodeSelector }}
nodeSelector: {{- toYaml . | nindent 8 }}
{{- end }}
Expand Down
5 changes: 5 additions & 0 deletions charts/kyuubi/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,11 @@ server:
env: []
envFrom: []

# Additional volumes for Kyuubi pod (templated)
volumes: []
# Additional volumeMounts for Kyuubi container (templated)
volumeMounts: []

service:
type: NodePort
# The default port limit of kubernetes is 30000-32767
Expand Down

0 comments on commit e9b713f

Please sign in to comment.