fix(alerting): alerte fiable sur l'échec de la vidéo quotidienne du brief #12

Merged
arcodange merged 1 commits from arcodange/alert-brief-video into main 2026-07-23 22:06:17 +02:00
Owner

L'alerte ProspectionBriefNotSent (prospection_brief_telegram_pushed == 0, severity info) avait trois défauts pour attraper un échec de production de la vidéo :

  1. Elle ne couvrait que la livraisonprospection_brief_rendered == 0 (rendu vidéo KO) n'alertait nulle part en dehors du ProspectionStepError générique (info) ;
  2. Faux positif sur skip volontaire : metrics.py émet brief_rendered=0 aussi quand l'étape est sautée (cadence/kill-switch) — seul prospection_step_status{step="brief"} distingue skip(2) d'erreur(0) ;
  3. En-tête du groupe obsolète (« livraison AM pas câblée » — testée live le 10/07 : firing 🔥 + resolved reçus sur Telegram).

Remplacée par deux règles warning, gardées contre le skip :

Alerte Expr Sens
ProspectionBriefFailed rendered == 0 and on(job) step_status{step="brief"} != 2 la production de la vidéo a échoué
ProspectionBriefNotSent pushed == 0 and rendered == 1 vidéo produite mais pas livrée sur Telegram

Validé en live sur Prometheus avant commit : requête témoin (rendered==1 and on(job) step_status{brief}!=2) → exactement 1 série (le match vectoriel fonctionne) ; les deux expressions d'alerte → vides sur l'état sain du jour. ArgoCD (app tools, sync auto) recharge les règles au merge.

🤖 Generated with Claude Code

L'alerte `ProspectionBriefNotSent` (`prospection_brief_telegram_pushed == 0`, severity **info**) avait trois défauts pour attraper un échec de **production** de la vidéo : 1. Elle ne couvrait que la **livraison** — `prospection_brief_rendered == 0` (rendu vidéo KO) n'alertait nulle part en dehors du `ProspectionStepError` générique (info) ; 2. **Faux positif sur skip volontaire** : `metrics.py` émet `brief_rendered=0` aussi quand l'étape est sautée (cadence/kill-switch) — seul `prospection_step_status{step="brief"}` distingue skip(`2`) d'erreur(`0`) ; 3. En-tête du groupe obsolète (« livraison AM pas câblée » — testée live le 10/07 : firing 🔥 + resolved ✅ reçus sur Telegram). **Remplacée par deux règles `warning`, gardées contre le skip :** | Alerte | Expr | Sens | |---|---|---| | `ProspectionBriefFailed` | `rendered == 0 and on(job) step_status{step="brief"} != 2` | la **production** de la vidéo a échoué | | `ProspectionBriefNotSent` | `pushed == 0 and rendered == 1` | vidéo produite mais **pas livrée** sur Telegram | **Validé en live sur Prometheus** avant commit : requête témoin (`rendered==1 and on(job) step_status{brief}!=2`) → exactement 1 série (le match vectoriel fonctionne) ; les deux expressions d'alerte → vides sur l'état sain du jour. ArgoCD (app `tools`, sync auto) recharge les règles au merge. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
arcodange added 1 commit 2026-07-23 22:06:08 +02:00
fix(alerting): alerte fiable sur l'échec de la vidéo quotidienne du brief
Helm Charts / Detect changed charts (push) Successful in 2m50s
Helm Charts / Detect changed charts (pull_request) Successful in 18s
Helm Charts / Library charts tool (push) Has been skipped
Helm Charts / Application charts pgcat (pull_request) Has been skipped
Helm Charts / Library charts tool (pull_request) Has been skipped
Helm Charts / Application charts pgcat (push) Has been skipped
8e236230d3
ProspectionBriefNotSent (pushed==0, severity info) ne couvrait que la
livraison et déclenchait à tort quand l'étape brief était volontairement
sautée (metrics.py émet brief_rendered=0 aussi sur skip — seul
step_status{step="brief"} distingue skip(2) d'erreur(0)).

Remplacé par deux règles warning, gardées contre le skip :
- ProspectionBriefFailed : rendered==0 AND step_status{brief}!=2 → la
  PRODUCTION de la vidéo a échoué (TTS/ffmpeg/PIL ou erreur amont) ;
- ProspectionBriefNotSent : pushed==0 AND rendered==1 → vidéo produite
  mais PAS livrée sur Telegram (token/chat_id/API).

Expressions validées en live sur Prometheus (match on(job) prouvé par
requête témoin, les deux exprs vides sur l'état sain du jour). En-tête du
groupe mis à jour : la livraison AM→Telegram est câblée et testée.

Co-Authored-By: Claude Opus 4.8 <[email protected]>
arcodange merged commit bc62304bc3 into main 2026-07-23 22:06:17 +02:00
arcodange deleted branch arcodange/alert-brief-video 2026-07-23 22:06:19 +02:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: arcodange-org/tools#12