Skip to content

Commit

Permalink
fix(metricsPort): use httpPort for metrics, as they are reachable ove…
Browse files Browse the repository at this point in the history
…r 9200 if prom-exporter is used

Signed-off-by: Toni Tauro <toni.tauro@adfinis.com>
  • Loading branch information
eyenx committed Dec 13, 2024
1 parent 25896e6 commit f3409a4
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion charts/opensearch/templates/serviceMonitor.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ spec:
matchLabels:
{{- include "opensearch.selectorLabels" . | nindent 6 }}
endpoints:
- port: {{ .Values.service.metricsPortName | default "metrics" }}
- port: {{ .Values.service.httpPortName | default "http" }}
interval: {{ .Values.serviceMonitor.interval }}
path: {{ .Values.serviceMonitor.path }}
{{- end }}
2 changes: 1 addition & 1 deletion charts/opensearch/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -281,7 +281,7 @@ enableServiceLinks: true
protocol: https
httpPort: 9200
transportPort: 9300
metricsPort: 9200
metricsPort: 9600
httpHostPort: ""
transportHostPort: ""

Expand Down

0 comments on commit f3409a4

Please sign in to comment.