Skip to content

Commit

Permalink
Use correct helm values for observability metrics (#245)
Browse files Browse the repository at this point in the history
  • Loading branch information
nrfox authored Feb 9, 2024
1 parent aafc65e commit 9ee28b7
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
8 changes: 4 additions & 4 deletions kiali-server/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,9 +26,9 @@ spec:
{{- end }}
annotations:
checksum/config: {{ include (print $.Template.BasePath "/configmap.yaml") . | sha256sum }}
{{- if .Values.server.metrics_enabled }}
{{- if .Values.server.observability.metrics.enabled }}
prometheus.io/scrape: "true"
prometheus.io/port: {{ .Values.server.metrics_port | quote }}
prometheus.io/port: {{ .Values.server.observability.metrics.port | quote }}
{{- else }}
prometheus.io/scrape: "false"
prometheus.io/port: ""
Expand Down Expand Up @@ -75,9 +75,9 @@ spec:
ports:
- name: api-port
containerPort: {{ .Values.server.port | default 20001 }}
{{- if .Values.server.metrics_enabled }}
{{- if .Values.server.observability.metrics.enabled }}
- name: http-metrics
containerPort: {{ .Values.server.metrics_port | default 9090 }}
containerPort: {{ .Values.server.observability.metrics.port | default 9090 }}
{{- end }}
readinessProbe:
httpGet:
Expand Down
4 changes: 2 additions & 2 deletions kiali-server/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,11 +34,11 @@ spec:
{{- end }}
protocol: TCP
port: {{ .Values.server.port }}
{{- if .Values.server.metrics_enabled }}
{{- if .Values.server.observability.metrics.enabled }}
- name: http-metrics
appProtocol: http
protocol: TCP
port: {{ .Values.server.metrics_port }}
port: {{ .Values.server.observability.metrics.port }}
{{- end }}
selector:
{{- include "kiali-server.selectorLabels" . | nindent 4 }}
Expand Down

0 comments on commit 9ee28b7

Please sign in to comment.