Skip to content

Commit 118b2bb

Browse files
committed
fix(helm): add ipForwarding monitor and prometheusRule to values.yaml.template
CI uses envsubst to generate values.yaml from the template file. The template was missing the monitors.ipForwarding section and the entire prometheusRule section, causing helm lint to fail with "nil pointer evaluating interface {}.enabled".
1 parent 17978c2 commit 118b2bb

File tree

1 file changed

+78
-0
lines changed

1 file changed

+78
-0
lines changed

helm/node-doctor/values.yaml.template

Lines changed: 78 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -133,6 +133,76 @@ serviceMonitor:
133133
# Relabelings
134134
relabelings: []
135135

136+
# Prometheus PrometheusRule configuration (for Prometheus Operator alerting)
137+
prometheusRule:
138+
# Enable PrometheusRule creation
139+
enabled: false
140+
# Namespace where PrometheusRule will be created (defaults to release namespace)
141+
namespace: ""
142+
# Additional labels for PrometheusRule (e.g., for Prometheus Operator discovery)
143+
labels: {}
144+
# Additional custom rules to add
145+
additionalRules: []
146+
147+
# Critical alerts - require immediate attention
148+
critical:
149+
enabled: true
150+
cniConfigInvalid:
151+
for: 5m
152+
labels: {}
153+
annotations: {}
154+
cniUnhealthy:
155+
for: 5m
156+
labels: {}
157+
annotations: {}
158+
networkPartitioned:
159+
for: 2m
160+
labels: {}
161+
annotations: {}
162+
kubeletUnhealthy:
163+
for: 5m
164+
labels: {}
165+
annotations: {}
166+
readOnlyFilesystem:
167+
for: 1m
168+
labels: {}
169+
annotations: {}
170+
ipForwardingDisabled:
171+
for: 2m
172+
labels: {}
173+
annotations: {}
174+
175+
# Warning alerts - should be investigated soon
176+
warning:
177+
enabled: true
178+
dnsResolutionFailed:
179+
for: 5m
180+
dnsLatencyHigh:
181+
for: 10m
182+
memoryPressure:
183+
for: 10m
184+
diskPressure:
185+
for: 10m
186+
networkDegraded:
187+
for: 10m
188+
highPeerLatency:
189+
for: 10m
190+
thresholdMs: 100
191+
lowPeerConnectivity:
192+
for: 5m
193+
thresholdPercent: 90
194+
apiServerLatencyHigh:
195+
for: 10m
196+
197+
# Informational alerts - for awareness
198+
info:
199+
enabled: true
200+
highEventRate:
201+
for: 5m
202+
threshold: 10
203+
noMetrics:
204+
for: 10m
205+
136206
# Health probe configuration
137207
probes:
138208
liveness:
@@ -276,6 +346,14 @@ monitors:
276346
readOnlyDetection: true
277347
ioHealthCheck: true
278348

349+
ipForwarding:
350+
enabled: true
351+
interval: 30s
352+
timeout: 5s
353+
checkIPv4: true
354+
checkIPv6: true
355+
checkPerInterface: false
356+
279357
# Exporters configuration
280358
exporters:
281359
kubernetes:

0 commit comments

Comments
 (0)