[incubator-heron] 01/01: Adding option to switch the apiserver service between NodePort and ClusterIP

[ww...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

wwong pushed a commit to branch helm_apiserver_service
in repository https://gitbox.apache.org/repos/asf/incubator-heron.git

commit b8388f32c45493f22d0579b6c957d3887b43c5ad
Author: Windham Wong <wi...@stormeye.io>
AuthorDate: Mon Oct 25 16:03:20 2021 +0800

    Adding option to switch the apiserver service between NodePort and ClusterIP
---
 deploy/kubernetes/helm/templates/tools.yaml | 5 +++++
 deploy/kubernetes/helm/values.yaml.template | 3 +++
 2 files changed, 8 insertions(+)

diff --git a/deploy/kubernetes/helm/templates/tools.yaml b/deploy/kubernetes/helm/templates/tools.yaml
index 0a6fe15..a78cba5 100644
--- a/deploy/kubernetes/helm/templates/tools.yaml
+++ b/deploy/kubernetes/helm/templates/tools.yaml
@@ -281,7 +281,12 @@ spec:
     - port: 9000
       targetPort: 9000
       protocol: TCP
+  {{- if eq .Values.apiserver.NodePort true }}
   type: NodePort
+  {{- else }}
+  type: ClusterIP
+  {{- end }}
+
   selector:
     app: {{ .Release.Name }}-tools
     release: {{ .Release.Name }}
diff --git a/deploy/kubernetes/helm/values.yaml.template b/deploy/kubernetes/helm/values.yaml.template
index 9484d98..46cb7f1 100644
--- a/deploy/kubernetes/helm/values.yaml.template
+++ b/deploy/kubernetes/helm/values.yaml.template
@@ -43,6 +43,9 @@ heron:
   # set to `null` to remove the use of base_url
   url: "-"
 
+apiserver:
+  NodePort: true
+
 # Can be EQUAL_TO_LIMIT or NOT_SET
 topologyResourceRequestMode: EQUAL_TO_LIMIT