You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kyuubi.apache.org by ch...@apache.org on 2022/12/30 02:00:13 UTC
[kyuubi] branch master updated: [KYUUBI #4043] [K8S][HELM] Add volumes and volumeMounts support to helm chart
This is an automated email from the ASF dual-hosted git repository.
chengpan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kyuubi.git
The following commit(s) were added to refs/heads/master by this push:
new e9b713f4e [KYUUBI #4043] [K8S][HELM] Add volumes and volumeMounts support to helm chart
e9b713f4e is described below
commit e9b713f4e9ca2d7346b5361dd94b751ee76b1322
Author: dnskr <dn...@gmail.com>
AuthorDate: Fri Dec 30 10:00:03 2022 +0800
[KYUUBI #4043] [K8S][HELM] Add volumes and volumeMounts support to helm 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
a332b92a [dnskr] [K8S][HELM] Add volumes and volumeMounts support to helm chart
Authored-by: dnskr <dn...@gmail.com>
Signed-off-by: Cheng Pan <ch...@apache.org>
---
charts/kyuubi/templates/kyuubi-deployment.yaml | 6 ++++++
charts/kyuubi/values.yaml | 5 +++++
2 files changed, 11 insertions(+)
diff --git a/charts/kyuubi/templates/kyuubi-deployment.yaml b/charts/kyuubi/templates/kyuubi-deployment.yaml
index 7560c0bdb..e0ad5bb78 100644
--- a/charts/kyuubi/templates/kyuubi-deployment.yaml
+++ b/charts/kyuubi/templates/kyuubi-deployment.yaml
@@ -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 }}
diff --git a/charts/kyuubi/values.yaml b/charts/kyuubi/values.yaml
index 9b32a5f20..cc8bf048a 100644
--- a/charts/kyuubi/values.yaml
+++ b/charts/kyuubi/values.yaml
@@ -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