Skip to content

Commit

Permalink
feat: added ability to define the nodePort in the trino service
Browse files Browse the repository at this point in the history
  • Loading branch information
dantonbertuol committed Aug 14, 2024
1 parent 8bff35d commit 651fdf8
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 0 deletions.
1 change: 1 addition & 0 deletions charts/trino/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -309,6 +309,7 @@ Fast distributed SQL query engine for big data analytics that helps you explore
* `service.annotations` - object, default: `{}`
* `service.type` - string, default: `"ClusterIP"`
* `service.port` - int, default: `8080`
* `service.nodePort` - int, default: `""`
* `auth` - object, default: `{}`

Available authentication methods.
Expand Down
3 changes: 3 additions & 0 deletions charts/trino/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,9 @@ spec:
targetPort: http
protocol: TCP
name: http
{{- if .Values.service.nodePort }}
nodePort: {{ .Values.service.nodePort }}
{{- end }}
{{- if .Values.jmx.exporter.enabled }}
- port: {{ .Values.jmx.exporter.port }}
targetPort: jmx-exporter
Expand Down
2 changes: 2 additions & 0 deletions charts/trino/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -348,6 +348,8 @@ service:
annotations: {}
type: ClusterIP
port: 8080
# service.nodePort -- The port the service listens on the host, for NodePort type. If not set, Kubernetes will [allocate a port automatically](https://kubernetes.io/docs/concepts/services-networking/service/#nodeport-custom-port).
nodePort: ""

auth: {}
# auth -- Available authentication methods.
Expand Down
3 changes: 3 additions & 0 deletions test-values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,9 @@ additionalConfigProperties:
service:
annotations:
custom/name: value
type: NodePort
port: 8080
nodePort: 30080

auth:
# created using htpasswd -B -C 10 password.db admin
Expand Down

0 comments on commit 651fdf8

Please sign in to comment.