g2/enervision

Watch

Fork 0

[44] Rejeu runbook collecteur et correction #191

Merged

lenaic merged 2 commits from justine/44-rejeu-collecteur into develop

2026-09-08 10:24:25 +00:00

justine commented

2026-09-08 10:02:05 +00:00

Member

Ce que ça change

Rejeu et correction du runbook

Closes #

Preuve

Relecture

Un pair a relu et laissé un commentaire, même court
Ses remarques sont traitées, ou une réponse explique pourquoi elles ne le sont pas

Ce qui suit le code

docs/runbooks/ mis à jour, un geste d'exploitation a changé
docs/adr/ complété, une décision structurante a été prise
docs/journal.md complété, un incident a été rencontré
Une nouvelle variable d'environnement est apparue, elle est dans .env.example

## Ce que ça change Rejeu et correction du runbook Closes # ## Preuve ![image](/attachments/5e5f9de3-896a-4a9b-ab32-95f21a2fda45) ![image](/attachments/68c6cbbe-05ba-47b1-b3f5-d9ad9f3dc8d9) ![image](/attachments/eaacf5e2-3f7a-42f8-b9ce-c55e0fbe0d71) ## Relecture - [ ] Un pair a relu et laissé un commentaire, même court - [ ] Ses remarques sont traitées, ou une réponse explique pourquoi elles ne le sont pas ## Ce qui suit le code  - [ ] `docs/runbooks/` mis à jour, un geste d'exploitation a changé - [x] `docs/adr/` complété, une décision structurante a été prise - [ ] `docs/journal.md` complété, un incident a été rencontré - [ ] Une nouvelle variable d'environnement est apparue, elle est dans `.env.example`

image.png

19 KiB

image.png

19 KiB

image.png

17 KiB

image.png

112 KiB

justine added 1 commit

2026-09-08 10:02:06 +00:00

[44] Rejeu runbook collecteur et correction

Intégration / Contrôles statiques du dépôt (pull_request) Successful in 10s

Details

Intégration / Tableau de bord — dépendances, tests et construction (pull_request) Successful in 38s

Details

Intégration / Workflows — lint et audit de sécurité (pull_request) Successful in 20s

Details

Infra Ansible / Playbooks Ansible valides (pull_request) Successful in 1m42s

Details

Intégration / Python — qualité, tests et dépendances (pull_request) Successful in 5m44s

Details

41c397d844

justine self-assigned this

2026-09-08 10:02:14 +00:00

justine requested reviews from gabriel, lenaic

2026-09-08 10:02:30 +00:00

lenaic approved these changes

2026-09-08 10:05:31 +00:00

lenaic left a comment

Approuvée. Le défaut est réel, vérifié dans le lanceur :

services/collector/bin/rattrapage-readings.sh
  12:  RACINE="${ENERVISION_RACINE:-/opt/enervision}"
  28:  cd "$RACINE/services/collector"

Le dépôt est cloné dans .repo, donc le défaut pointe sur un répertoire qui n'existe pas. La commande publiée ne pouvait pas marcher, et ça n'avait jamais été vu parce que la crontab posée par le rôle app porte la variable : seul un rejeu à la main tombe dessus.

C'est exactement ce que le #44 cherche. Une procédure rejouée par quelqu'un qui ne l'a pas écrite, sans poser de question, et le blocage consigné plutôt que contourné. Ton observation sur le message d'erreur est la bonne : cd: can't cd to /opt/enervision/services/collector nomme le répertoire absent sans dire lequel est le bon, et le manuel donnait pourtant .repo deux lignes plus haut. C'est le genre d'incohérence qu'on ne voit qu'en exécutant.

Le journal des rejeux est une bonne idée et je le reprends : je vais poser le même tableau dans reprise.md et deploiement.md, dont j'écris les sections d'échec en #190. « Ce qui a bloqué est le livrable, pas la réussite » est la bonne phrase, je la garde telle quelle.

Deux détails que je ne bloque pas et que tu peux corriger si tu repasses : Closes # est resté vide, et la case cochée est docs/adr/ alors que le changement est un runbook. Rien qui empêche de fusionner.

Approuvée. Le défaut est réel, vérifié dans le lanceur : ``` services/collector/bin/rattrapage-readings.sh 12: RACINE="${ENERVISION_RACINE:-/opt/enervision}" 28: cd "$RACINE/services/collector" ``` Le dépôt est cloné dans `.repo`, donc le défaut pointe sur un répertoire qui n'existe pas. La commande publiée ne pouvait pas marcher, et ça n'avait jamais été vu parce que la crontab posée par le rôle `app` porte la variable : seul un rejeu à la main tombe dessus. **C'est exactement ce que le #44 cherche.** Une procédure rejouée par quelqu'un qui ne l'a pas écrite, sans poser de question, et le blocage consigné plutôt que contourné. Ton observation sur le message d'erreur est la bonne : `cd: can't cd to /opt/enervision/services/collector` nomme le répertoire absent sans dire lequel est le bon, et le manuel donnait pourtant `.repo` deux lignes plus haut. C'est le genre d'incohérence qu'on ne voit qu'en exécutant. Le journal des rejeux est une bonne idée et je le reprends : je vais poser le même tableau dans `reprise.md` et `deploiement.md`, dont j'écris les sections d'échec en #190. « Ce qui a bloqué est le livrable, pas la réussite » est la bonne phrase, je la garde telle quelle. Deux détails que je ne bloque pas et que tu peux corriger si tu repasses : `Closes #` est resté vide, et la case cochée est `docs/adr/` alors que le changement est un runbook. Rien qui empêche de fusionner.

lenaic added 1 commit

2026-09-08 10:18:13 +00:00

Merge branch 'develop' into justine/44-rejeu-collecteur

Intégration / Tableau de bord — dépendances, tests et construction (pull_request) Successful in 39s

Details

Intégration / Contrôles statiques du dépôt (pull_request) Successful in 6s

Details

Intégration / Workflows — lint et audit de sécurité (pull_request) Successful in 17s

Details

Intégration / Python — qualité, tests et dépendances (pull_request) Successful in 5m36s

Details

62d035ff4b