This is an automated email from the ASF dual-hosted git repository.
zhouky pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-celeborn.git
The following commit(s) were added to refs/heads/main by this push:
new 859d021ee [CELEBORN-644][HELM] Support Helm Deploy Celeborn with
HostNetwork And DnsPolicy
859d021ee is described below
commit 859d021eeb90e8bac6596a1aae35519d9ebc78e1
Author: zwangsheng <[email protected]>
AuthorDate: Fri Jun 16 17:59:39 2023 +0800
[CELEBORN-644][HELM] Support Helm Deploy Celeborn with HostNetwork And
DnsPolicy
### What changes were proposed in this pull request?
Support service type `NodePort` in `values.yaml`
### Why are the changes needed?
For Kubernetes IT, should start celeborn cluster in minikube with nodePort
for discover
### Does this PR introduce _any_ user-facing change?
Yes
### How was this patch tested?
CI Test & Local Test
Closes #1554 from zwangsheng/CELEBORN-644.
Authored-by: zwangsheng <[email protected]>
Signed-off-by: zky.zhoukeyong <[email protected]>
---
charts/celeborn/templates/master-statefulset.yaml | 2 ++
charts/celeborn/templates/worker-statefulset.yaml | 2 ++
charts/celeborn/values.yaml | 3 +++
tests/kubernetes-it/docker/helm/values.yaml | 2 ++
4 files changed, 9 insertions(+)
diff --git a/charts/celeborn/templates/master-statefulset.yaml
b/charts/celeborn/templates/master-statefulset.yaml
index 974771005..be25769fc 100644
--- a/charts/celeborn/templates/master-statefulset.yaml
+++ b/charts/celeborn/templates/master-statefulset.yaml
@@ -71,6 +71,8 @@ spec:
securityContext:
{{- toYaml . | nindent 8 }}
{{- end }}
+ hostNetwork: {{ .Values.hostNetwork | default false }}
+ dnsPolicy: {{ .Values.dnsPolicy | default "ClusterFirst" }}
initContainers:
{{- $dirs := .Values.volumes.master }}
{{- if eq "hostPath" (index $dirs 0).type }}
diff --git a/charts/celeborn/templates/worker-statefulset.yaml
b/charts/celeborn/templates/worker-statefulset.yaml
index 1d758fc8d..0a4ab5756 100644
--- a/charts/celeborn/templates/worker-statefulset.yaml
+++ b/charts/celeborn/templates/worker-statefulset.yaml
@@ -71,6 +71,8 @@ spec:
securityContext:
{{- toYaml . | nindent 8 }}
{{- end }}
+ hostNetwork: {{ .Values.hostNetwork | default false }}
+ dnsPolicy: {{ .Values.dnsPolicy | default "ClusterFirst" }}
initContainers:
{{- $dirs := .Values.volumes.worker }}
{{- if eq "hostPath" (index $dirs 0).type }}
diff --git a/charts/celeborn/values.yaml b/charts/celeborn/values.yaml
index 932fefaea..741efc3cc 100644
--- a/charts/celeborn/values.yaml
+++ b/charts/celeborn/values.yaml
@@ -33,6 +33,9 @@ masterReplicas: 3
# worker replicas set on demand, should less than node number
workerReplicas: 5
+hostNetwork: false
+dnsPolicy: ClusterFirst
+
securityContext:
runAsUser: 10006
runAsGroup: 10006
diff --git a/tests/kubernetes-it/docker/helm/values.yaml
b/tests/kubernetes-it/docker/helm/values.yaml
index f2b88c4d2..9d36ab62e 100644
--- a/tests/kubernetes-it/docker/helm/values.yaml
+++ b/tests/kubernetes-it/docker/helm/values.yaml
@@ -26,6 +26,8 @@ imagePullSecrets: {}
masterReplicas: 1
workerReplicas: 1
+hostNetwork: true
+dnsPolicy: ClusterFirstWithHostNet
securityContext:
runAsUser: 10006
runAsGroup: 10006
