Skip to content

Commit

Permalink
Changes for the helm chart added, specifically allowing users to opt-…
Browse files Browse the repository at this point in the history
…in for access to the Go Plugin Filter Metrics

Signed-off-by: w-h37 <[email protected]>
  • Loading branch information
w-h37 committed Aug 1, 2024
1 parent 70cfbfb commit 4103bc1
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 0 deletions.
6 changes: 6 additions & 0 deletions charts/emissary-ingress/templates/admin-service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,12 @@ spec:
{{- if (and (eq .Values.adminService.type "NodePort") (not (empty .Values.adminService.nodePort))) }}
nodePort: {{ int .Values.adminService.nodePort }}
{{- end }}
{{- if .Values.adminService.goPluginMetricsPort}}
- port: {{ .Values.adminService.goPluginMetricsPort }}
targetPort: 8878
protocol: TCP
name: go-plugin-filter-metrics
{{- end }}
- port: {{ .Values.adminService.snapshotPort }}
targetPort: 8005
protocol: TCP
Expand Down
8 changes: 8 additions & 0 deletions charts/emissary-ingress/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -215,7 +215,15 @@ spec:
{{- end}}
- name: admin
containerPort: {{ .Values.adminService.port }}
{{- if .Values.adminService.goPluginMetricsPort}}
- name: go-plugin-metrics-port
containerPort: {{ .Values.adminService.goPluginMetricsPort}}
{{- end}}
env:
{{- if .Values.adminService.goPluginMetricsPort}}
- name: ENABLE_PLUGIN_FILTER_METRICS
value: "true"
{{- end }}
{{- if .Values.prometheusExporter.enabled }}
- name: STATSD_ENABLED
value: "true"
Expand Down
1 change: 1 addition & 0 deletions charts/emissary-ingress/values.yaml.in
Original file line number Diff line number Diff line change
Expand Up @@ -208,6 +208,7 @@ adminService:
type: ClusterIP
port: 8877
snapshotPort: 8005
goPluginMetricsPort:
# If explicit NodePort for admin service is required
nodePort:
# Annotations to apply to Ambassador admin service
Expand Down

0 comments on commit 4103bc1

Please sign in to comment.