service: seperate coordinator and worker service

Signed-off-by: Parsa Yousefi <[email protected]>

Author: parsa97

charts/trino/templates/service-coordinator.yaml

@@ -1,4 +1,5 @@
 {{- $coordinatorJmx := merge .Values.jmx.coordinator (omit .Values.jmx "coordinator" "worker") -}}
+{{- $svc := .Values.service | default .Values.serviceCoordinator }}
 apiVersion: v1
 kind: Service
 metadata:
@@ -8,16 +9,16 @@ metadata:
     {{- include "trino.labels" . | nindent 4 }}
     app.kubernetes.io/component: coordinator
   annotations:
-    {{- toYaml .Values.service.annotations | nindent 4 }}
+    {{- toYaml $svc.annotations | nindent 4 }}
 spec:
-  type: {{ .Values.service.type }}
+  type: {{ $svc.type }}
   ports:
-    - port: {{ .Values.service.port }}
+    - port: {{ $svc.port }}
       targetPort: http
       protocol: TCP
       name: http
-      {{- if .Values.service.nodePort }}
-      nodePort: {{ .Values.service.nodePort }}
+      {{- if $svc.nodePort }}
+      nodePort: {{ $svc.nodePort }}
       {{- end }}
     {{- if .Values.server.config.https.enabled }}
     - port: {{ .Values.server.config.https.port }}

charts/trino/templates/service-worker.yaml

@@ -1,3 +1,4 @@
+{{- $svc := .Values.service | default .Values.serviceWorker }}
 {{- $workerJmx := merge .Values.jmx.worker (omit .Values.jmx "coordinator" "worker") -}}
 apiVersion: v1
 kind: Service
@@ -8,11 +9,11 @@ metadata:
     {{- include "trino.labels" . | nindent 4 }}
     app.kubernetes.io/component: worker
   annotations:
-    {{- toYaml .Values.service.annotations | nindent 4 }}
+    {{- toYaml $svc.annotations | nindent 4 }}
 spec:
   clusterIP: None
   ports:
-    - port: {{ .Values.service.port }}
+    - port: {{ $svc.port }}
       targetPort: http
       protocol: TCP
       name: http

charts/trino/values.yaml

@@ -486,6 +486,24 @@ shareProcessNamespace:
   coordinator: false
   worker: false
 
+# serviceCoordinator:
+#   annotations: {}
+#   type: ClusterIP
+#   port: 8080
+#   # service.nodePort -- The port the service listens on the host, for the `NodePort` type. If not set, Kubernetes will
+#   # [allocate a port
+#   # automatically](https://kubernetes.io/docs/concepts/services-networking/service/#nodeport-custom-port).
+#   nodePort: ""
+
+# serviceWorker:
+#   annotations: {}
+#   type: ClusterIP
+#   port: 8080
+#   # service.nodePort -- The port the service listens on the host, for the `NodePort` type. If not set, Kubernetes will
+#   # [allocate a port
+#   # automatically](https://kubernetes.io/docs/concepts/services-networking/service/#nodeport-custom-port).
+#   nodePort: ""
+
 service:
   annotations: {}
   type: ClusterIP

tests/trino/test-values.yaml

@@ -44,6 +44,11 @@ additionalConfigProperties:
   - http-server.authentication.allow-insecure-over-http=true
   - http-server.process-forwarded=true
 
+serviceCoordinator:
+  annotations:
+    custom/worker: value
+  port: 8080
+
 service:
   annotations:
     custom/name: value