Skip to content

Commit 0fbd47d

Browse files
committed
fix(base-cluster/monitoring): grafana-tempo-distributed would need s3
Therefore we switch to the singleBinary chart. Without this change the compactor doesn't work; meaning the traces are never cleaned up...
1 parent b50ea98 commit 0fbd47d

File tree

3 files changed

+32
-32
lines changed

3 files changed

+32
-32
lines changed

charts/base-cluster/templates/monitoring/alloy-gateway.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ spec:
9898
9999
otelcol.exporter.otlp "tempo" {
100100
client {
101-
endpoint = "grafana-tempo-distributor:4317"
101+
endpoint = "grafana-tempo:4317"
102102
103103
tls {
104104
insecure = true

charts/base-cluster/templates/monitoring/tracing/grafana-tempo.yaml

Lines changed: 30 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -9,51 +9,51 @@ metadata:
99
app.kubernetes.io/part-of: monitoring
1010
spec:
1111
chart:
12-
spec: {{- include "base-cluster.helm.chartSpec" (dict "repo" "grafana" "chart" "tempo-distributed" "context" $) | nindent 6 }}
12+
spec: {{- include "base-cluster.helm.chartSpec" (dict "repo" "grafana" "chart" "tempo" "context" $) | nindent 6 }}
1313
interval: 1h
1414
driftDetection:
1515
mode: enabled
1616
dependsOn:
1717
- name: kube-prometheus-stack
1818
namespace: monitoring
1919
values:
20-
{{- if .Values.global.imageRegistry }}
21-
global:
22-
image:
23-
registry: {{ $.Values.global.imageRegistry }}
24-
{{- end }}
25-
ingester: {{- include "common.resourcesWithPreset" .Values.monitoring.tracing.ingester | nindent 6 }}
26-
persistentVolumeClaimRetentionPolicy:
27-
enabled: true
28-
whenDeleted: Delete
29-
whenScaled: Retain
30-
replicas: &replicas 1
31-
config:
32-
replication_factor: *replicas
33-
persistence:
34-
enabled: true
35-
size: {{ .Values.monitoring.tracing.ingester.persistence.size }}
3620
tempo:
21+
{{- with .Values.global.imageRegistry }}
22+
registry: {{ . }}
23+
{{- end }}
24+
resources: {{- include "common.resources" .Values.monitoring.tracing.resources | nindent 8 }}
3725
securityContext:
3826
privileged: false
3927
seLinuxOptions: {}
4028
seccompProfile:
4129
type: RuntimeDefault
42-
podSecurityContext:
43-
fsGroupChangePolicy: OnRootMismatch
44-
supplementalGroups: []
45-
sysctls: []
30+
receivers:
31+
otlp:
32+
protocols:
33+
grpc:
34+
endpoint: "0.0.0.0:4317"
35+
reportingEnabled: false
36+
37+
persistentVolumeClaimRetentionPolicy:
38+
enabled: true
39+
whenDeleted: Delete
40+
whenScaled: Retain
41+
replicas: 1
42+
retention: 48h
43+
persistence:
44+
enabled: true
45+
size: {{ .Values.monitoring.tracing.ingester.persistence.size }}
46+
securityContext:
47+
fsGroupChangePolicy: OnRootMismatch
48+
supplementalGroups: []
49+
sysctls: []
4650
metricsGenerator:
4751
enabled: true
48-
traces:
49-
otlp:
50-
grpc:
51-
enabled: true
52-
metaMonitoring:
53-
serviceMonitor:
54-
enabled: true
55-
labels:
56-
monitoring/provisioned-by: base-cluster
52+
remoteWriteUrl: "http://kube-prometheus-stack-prometheus:9090/api/v1/write"
53+
serviceMonitor:
54+
enabled: true
55+
additionalLabels:
56+
monitoring/provisioned-by: base-cluster
5757
---
5858
apiVersion: v1
5959
kind: ConfigMap

charts/base-cluster/values.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ global:
111111
charts:
112112
loki: 6.46.0
113113
alloy: 1.5.0
114-
tempo-distributed: 1.57.0
114+
tempo: 1.24.1
115115
condition: "{{ and .Values.monitoring.prometheus.enabled (or .Values.monitoring.loki.enabled .Values.monitoring.tracing.enabled) }}"
116116
external-dns:
117117
url: https://kubernetes-sigs.github.io/external-dns

0 commit comments

Comments
 (0)