Skip to content

Commit e1d3d0c

Browse files
committed
refactor(tempo-distributed): include servicemonitor for (jaeger) tempo-query only if enabled
Currently the service monitor is added regardless of whether the old jaeger mode is enabled or not. Similar to the ports on the deployment and service, we only include the servicemonitor if `queryFrontend.query.enabled=true`. Additionally, the selector of the service monitor does not work anyway, because the service that includes the old ports has a different `app.kubernetes.io/component` label than the one created from the servicemonitor template Signed-off-by: Alexander Bartolomey <[email protected]>
1 parent e1145ac commit e1d3d0c

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed
Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,3 @@
1+
{{- if .Values.queryFrontend.query.enabled }}
12
{{- include "tempo.lib.serviceMonitor" (dict "ctx" $ "component" "tempo-query") }}
3+
{{- end }}

0 commit comments

Comments
 (0)