viktor/infra
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: bank sync alerts - remove {{ $labels.job }} that Helm provider silently drops [ci skip]

Browse source 
The Terraform Helm provider's YAML diff comparison silently ignores rules
containing {{ $labels.job }} in annotations, preventing the alerts from being
applied. Also syncs alerts to platform stack tpl.
This commit is contained in:
Viktor Barzin 2026-04-05 20:07:51 +03:00
parent f8daf7a245
commit 3cd560d4d9
2 changed files with 327 additions and 33 deletions
Download patch file Download diff file Expand all files Collapse all files

4
stacks/monitoring/modules/monitoring/prometheus_chart_values.tpl
View file

@ -1506,14 +1506,14 @@ serverFiles:
labels:
severity: warning
annotations:
summary: "Bank sync failing for {{ $labels.job }}. Accounts may need GoCardless reauthorization."
summary: "Bank sync failing. Accounts may need GoCardless reauthorization. Check Pushgateway for which instance."
- alert: BankSyncStale
expr: (time() - bank_sync_last_success_timestamp) > 172800
for: 1h
labels:
severity: warning
annotations:
summary: "Bank sync for {{ $labels.job }} has not succeeded in >48h. Check CronJob and account auth."
summary: "Bank sync has not succeeded in more than 48h. Check CronJob and account auth."
- alert: EmailRoundtripFailing
expr: email_roundtrip_success{job="email-roundtrip-monitor"} == 0
for: 90m