Add possibility to expose extra ports

FleischerT · Apr 11, 2023 · dd8aa52 · dd8aa52
1 parent 38370cb
commit dd8aa52
Show file tree

Hide file tree

Showing 5 changed files with 22 additions and 0 deletions.
diff --git a/charts/trino/README.md b/charts/trino/README.md
@@ -58,6 +58,7 @@ The following table lists the configurable parameters of the Trino chart and the
 | `coordinator.config.memory.heapHeadroomPerNode` |  | `""` |
 | `coordinator.config.query.maxMemoryPerNode` |  | `"1GB"` |
 | `coordinator.additionalJVMConfig` |  | `{}` |
+| `coordinator.additionalExposedPorts` |  | `{}` |
 | `coordinator.resources` |  | `{}` |
 | `coordinator.livenessProbe` |  | `{}` |
 | `coordinator.readinessProbe` |  | `{}` |
@@ -71,6 +72,7 @@ The following table lists the configurable parameters of the Trino chart and the
 | `worker.config.memory.heapHeadroomPerNode` |  | `""` |
 | `worker.config.query.maxMemoryPerNode` |  | `"1GB"` |
 | `worker.additionalJVMConfig` |  | `{}` |
+| `worker.additionalExposedPorts` |  | `{}` |
 | `worker.resources` |  | `{}` |
 | `worker.livenessProbe` |  | `{}` |
 | `worker.readinessProbe` |  | `{}` |

diff --git a/charts/trino/templates/deployment-coordinator.yaml b/charts/trino/templates/deployment-coordinator.yaml
@@ -87,6 +87,11 @@ spec:
             - name: http
               containerPort: {{ .Values.service.port }}
               protocol: TCP
+          {{- range $key, $value := .Values.coordinator.additionalExposedPorts }}
+            - name: {{ $value.name }}
+              containerPort: {{ $value.port }}
+              protocol: {{ $value.protocol }}
+          {{- end }}
           livenessProbe:
             httpGet:
               path: /v1/info

diff --git a/charts/trino/templates/deployment-worker.yaml b/charts/trino/templates/deployment-worker.yaml
@@ -57,6 +57,11 @@ spec:
             - name: http
               containerPort: {{ .Values.service.port }}
               protocol: TCP
+          {{- range $key, $value := .Values.worker.additionalExposedPorts }}
+            - name: {{ $value.name }}
+              containerPort: {{ $value.port }}
+              protocol: {{ $value.protocol }}
+          {{- end }}
           livenessProbe:
             httpGet:
               path: /v1/info

diff --git a/charts/trino/templates/service.yaml b/charts/trino/templates/service.yaml
@@ -14,6 +14,12 @@ spec:
       targetPort: http
       protocol: TCP
       name: http
+    {{- range $key, $value := .Values.coordinator.additionalExposedPorts }}
+    - port: {{ $value.servicePort }}
+      name: {{ $value.name }}
+      targetPort: {{ $value.port }}
+      protocol: {{ $value.protocol }}
+    {{- end }}
   selector:
     app: {{ template "trino.name" . }}
     release: {{ .Release.Name }}

diff --git a/charts/trino/values.yaml b/charts/trino/values.yaml
@@ -161,6 +161,8 @@ coordinator:
 
   additionalJVMConfig: {}
 
+  additionalExposedPorts: {}
+
   resources: {}
     # We usually recommend not to specify default resources and to leave this as a conscious
     # choice for the user. This also increases chances charts run on environments with little
@@ -210,6 +212,8 @@ worker:
 
   additionalJVMConfig: {}
 
+  additionalExposedPorts: {}
+
   resources: {}
     # We usually recommend not to specify default resources and to leave this as a conscious
     # choice for the user. This also increases chances charts run on environments with little