Skip to content

Commit

Permalink
Allow setting service annotations
Browse files Browse the repository at this point in the history
Enables the user to add annotations for the service resource so it can
be integrated with load balancers and other operators.

Signed-off-by: Robert Sturla <robertsturla@outlook.com>
  • Loading branch information
p5 authored and nineinchnick committed May 30, 2024
1 parent 77f15a6 commit bc12506
Show file tree
Hide file tree
Showing 4 changed files with 8 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 @@ -295,6 +295,7 @@ Fast distributed SQL query engine for big data analytics that helps you explore
A list of the Linux kernel capabilities that are dropped from every container. Valid values are listed at https://man7.org/linux/man-pages/man7/capabilities.7.html Ensure to remove the "CAP_" prefix which the kernel attaches to the names of permissions.
* `shareProcessNamespace.coordinator` - bool, default: `false`
* `shareProcessNamespace.worker` - bool, default: `false`
* `service.annotations` - object, default: `{}`
* `service.type` - string, default: `"ClusterIP"`
* `service.port` - int, default: `8080`
* `auth` - object, default: `{}`
Expand Down
2 changes: 2 additions & 0 deletions charts/trino/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@ metadata:
{{- if .Values.commonLabels }}
{{- tpl (toYaml .Values.commonLabels) . | nindent 4 }}
{{- end }}
annotations:
{{- toYaml .Values.service.annotations | nindent 4 }}
spec:
type: {{ .Values.service.type }}
ports:
Expand Down
1 change: 1 addition & 0 deletions charts/trino/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -332,6 +332,7 @@ shareProcessNamespace:
worker: false

service:
annotations: {}
type: ClusterIP
port: 8080

Expand Down
4 changes: 4 additions & 0 deletions test-values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,10 @@ additionalConfigProperties:
- http-server.authentication.allow-insecure-over-http=true
- http-server.process-forwarded=true

service:
annotations:
custom/name: value

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

0 comments on commit bc12506

Please sign in to comment.