Disable prom Istio when mergeMetrics

streamnative · Nov 21, 2024 · 66b8ba8 · 66b8ba8
1 parent 3346912
commit 66b8ba8
Show file tree

Hide file tree

Showing 6 changed files with 27 additions and 5 deletions.
diff --git a/charts/sn-platform/templates/_helpers.tpl b/charts/sn-platform/templates/_helpers.tpl
@@ -77,6 +77,15 @@ sidecar.istio.io/inject: "true"
 {{- end }}
 {{- end }}
 
+{{/*
+Create the monitoring template labels.
+*/}}
+{{- define "pulsar.monitoring.template.labels" -}}
+app: {{ template "pulsar.name" . }}
+release: {{ .Release.Name }}
+cluster: {{ template "pulsar.fullname" . }}
+{{- end }}
+
 {{/*
 Create the match labels.
 */}}

diff --git a/charts/sn-platform/templates/grafana/grafana-deployment.yaml b/charts/sn-platform/templates/grafana/grafana-deployment.yaml
@@ -24,7 +24,11 @@ spec:
   template:
     metadata:
       labels:
+        {{- if and .Values.istio.enabled .Values.istio.mergeMetrics }}
+        {{- include "pulsar.monitoring.template.labels" . | nindent 8 }}
+        {{- else }}
         {{- include "pulsar.template.labels" . | nindent 8 }}
+        {{- end }}
         component: {{ .Values.grafana.component }}
 {{- with .Values.grafana.labels }}
 {{ toYaml . | indent 8 }}

diff --git a/charts/sn-platform/templates/grafana/grafana-statefulset.yaml b/charts/sn-platform/templates/grafana/grafana-statefulset.yaml
@@ -25,7 +25,11 @@ spec:
   template:
     metadata:
       labels:
+        {{- if and .Values.istio.enabled .Values.istio.mergeMetrics }}
+        {{- include "pulsar.monitoring.template.labels" . | nindent 8 }}
+        {{- else }}
         {{- include "pulsar.template.labels" . | nindent 8 }}
+        {{- end }}
         component: {{ .Values.grafana.component }}
       annotations:
         checksum/config: {{ include (print $.Template.BasePath "/grafana/grafana-configmap.yaml") . | sha256sum }}

diff --git a/charts/sn-platform/templates/prometheus/prometheus-configmap.yaml b/charts/sn-platform/templates/prometheus/prometheus-configmap.yaml
@@ -52,8 +52,8 @@ data:
           names:
             - {{ template "pulsar.namespace" . }}
 {{- end }}
-{{- if .Values.istio.enabled }}
-{{- if or .Values.istio.migration .Values.istio.mergeMetrics }}
+{{- if and .Values.istio.enabled (not .Values.istio.mergeMetrics) }}
+{{- if .Values.istio.migration }}
       scheme: http
       enable_http2: false
 {{- else }}

diff --git a/charts/sn-platform/templates/prometheus/prometheus-statefulset.yaml b/charts/sn-platform/templates/prometheus/prometheus-statefulset.yaml
@@ -27,13 +27,17 @@ spec:
   template:
     metadata:
       labels:
+        {{- if and .Values.istio.enabled .Values.istio.mergeMetrics }}
+        {{- include "pulsar.monitoring.template.labels" . | nindent 8 }}
+        {{- else }}
         {{- include "pulsar.template.labels" . | nindent 8 }}
+        {{- end }}
         component: {{ .Values.prometheus.component }}
 {{- with .Values.prometheus.labels }}
 {{ toYaml . | indent 8 }}
 {{- end }}
       annotations:
-        {{- if .Values.istio.enabled }}
+        {{- if and .Values.istio.enabled (not .Values.istio.mergeMetrics ) }}
         # ref: https://istio.io/latest/docs/ops/integrations/prometheus/#tls-settings
         traffic.sidecar.istio.io/includeOutboundIPRanges: ""  # do not intercept any outbound traffic
         # configure an env variable `OUTPUT_CERTS` to write certificates to the given folder
@@ -154,7 +158,7 @@ spec:
           mountPath: /etc/config
         - name: "{{ template "pulsar.fullname" . }}-{{ .Values.prometheus.component }}-{{ .Values.prometheus.volumes.data.name }}"
           mountPath: /prometheus
-        {{- if .Values.istio.enabled }}
+        {{- if and .Values.istio.enabled (not .Values.istio.mergeMetrics ) }}
         - name: istio-certs
           mountPath: /etc/prom-certs/
         {{- end }}
@@ -163,7 +167,7 @@ spec:
       - name: config-volume
         configMap:
           name: "{{ template "pulsar.fullname" . }}-{{ .Values.prometheus.component }}"
-      {{- if .Values.istio.enabled }}
+      {{- if and .Values.istio.enabled (not .Values.istio.mergeMetrics ) }}
       - emptyDir:
           medium: Memory
         name: istio-certs

diff --git a/charts/sn-platform/values.yaml b/charts/sn-platform/values.yaml
@@ -2510,6 +2510,7 @@ istio:
   # istio labels used to inject sidecars if it's not `sidecar.istio.io/inject: "true"`
   labels: {}
   # mergeMetrics should be enabled if you want to scrape pulsar metrics from an external prometheus not running on Istio
+  # prometheus and grafana will not be Istio injected if mergeMetrics is enabled
   mergeMetrics: false
   gateway:
     # gateway selector if it's not `istio: ingressgateway`