Add Helm parameterisation for scheduler and Envoy service types (#520)

* Remove trailing whitespace from SCv2 component Helm chart values * Add Kustomize patch for Envoy service type in components Helm chart * Add default Helm value for Envoy service type * Add Kustomize patch for scheduler service type in components Helm chart * Add default Helm value for scheduler service type
SeldonIO · Oct 19, 2022 · c16f09b · c16f09b
1 parent 542c7ce
commit c16f09b
Show file tree

Hide file tree

Showing 5 changed files with 28 additions and 6 deletions.
diff --git a/k8s/helm-charts/seldon-core-v2-setup/templates/seldon-v2-components.yaml b/k8s/helm-charts/seldon-core-v2-setup/templates/seldon-v2-components.yaml
@@ -465,7 +465,7 @@ spec:
     targetPort: envoy-admin
   selector:
     app: seldon-envoy
-  type: LoadBalancer
+  type: '{{ .Values.envoy.service.type }}'
 ---
 apiVersion: v1
 kind: Service
@@ -523,7 +523,7 @@ spec:
     targetPort: dataflow
   selector:
     control-plane: seldon-scheduler
-  type: LoadBalancer
+  type: '{{ .Values.scheduler.service.type }}'
 ---
 apiVersion: apps/v1
 kind: Deployment

diff --git a/k8s/helm-charts/seldon-core-v2-setup/values.yaml b/k8s/helm-charts/seldon-core-v2-setup/values.yaml
@@ -62,7 +62,7 @@ security:
           crtPath: /tmp/certs/ddc/tls.crt
           caPath: /tmp/certs/ddc/ca.crt
           serverCaPath: /tmp/certs/dds/ca.crt
-  
+
 opentelemetry:
   endpoint: seldon-collector:4317
   enable: true
@@ -146,14 +146,18 @@ envoy:
     registry: docker.io
     repository: seldonio/seldon-envoy
     tag: latest
+  service:
+    type: LoadBalancer
 
 scheduler:
   image:
     pullPolicy: IfNotPresent
     registry: docker.io
     repository: seldonio/seldon-scheduler
     tag: latest
-
+  service:
+    type: LoadBalancer
+
 serverConfig:
   rclone:
     image:

diff --git a/k8s/helm-charts/seldon-core-v2-setup/values.yaml.template b/k8s/helm-charts/seldon-core-v2-setup/values.yaml.template
@@ -62,7 +62,7 @@ security:
           crtPath: /tmp/certs/ddc/tls.crt
           caPath: /tmp/certs/ddc/ca.crt
           serverCaPath: /tmp/certs/dds/ca.crt
-  
+
 opentelemetry:
   endpoint: seldon-collector:4317
   enable: true
@@ -146,14 +146,18 @@ envoy:
     registry: docker.io
     repository: seldonio/seldon-envoy
     tag: #TAG_VERSION_PLACEHOLDER#
+  service:
+    type: LoadBalancer
 
 scheduler:
   image:
     pullPolicy: IfNotPresent
     registry: docker.io
     repository: seldonio/seldon-scheduler
     tag: #TAG_VERSION_PLACEHOLDER#
-
+  service:
+    type: LoadBalancer
+
 serverConfig:
   rclone:
     image:

diff --git a/k8s/kustomize/helm-components/patch_envoy.yaml b/k8s/kustomize/helm-components/patch_envoy.yaml
@@ -30,3 +30,10 @@ spec:
                 name: '{{ .Values.security.controlplane.ssl.client.serverValidationSecret }}' 
                 key: ca.crt
                 optional: true 
+---
+apiVersion: v1
+kind: Service
+metadata:
+  name: seldon-mesh
+spec:
+  type: '{{ .Values.envoy.service.type }}'
diff --git a/k8s/kustomize/helm-components/patch_scheduler.yaml b/k8s/kustomize/helm-components/patch_scheduler.yaml
@@ -50,3 +50,10 @@ spec:
             value: '{{ .Values.security.envoy.ssl.downstream.server.caPath }}'
           - name: ENVOY_DOWNSTREAM_CLIENT_TLS_CA_LOCATION
             value: '{{ .Values.security.envoy.ssl.downstream.server.clientCaPath }}'
+---
+apiVersion: v1
+kind: Service
+metadata:
+  name: scheduler
+spec:
+  type: '{{ .Values.scheduler.service.type }}'