[release][medium] Attendre la libération réelle du verrou dans le test de redémarrage #83

New issue

Open

opened 2026-10-03 16:49:29 +00:00 by hephaistos · 0 comments

hephaistos commented

2026-10-03 16:49:29 +00:00

Owner

Revue de préparation publique

Branche de travail : release.
Code examiné : 6052defc07a8d85a26a62014a72d23c4855a4786.
Sévérité : medium. Domaine : tests.

Constat et impact

La suite parallèle échoue de façon intermittente au second start du
test de livraison bloquée: session-lock, session déjà active. Le même
test passe trois fois isolé et la suite complète passe avec -p=1.
Le résultat persistant peut être lu avant la fermeture du verrou.

Preuves

internal/acceptance/acceptance_test.go:1418-1424;
internal/session/orchestrator.go:188-189;
internal/cli/start_unix.go:189,306-309.

Travail recommandé

Synchroniser le test sur la terminaison/libération effective, puis
stresser la reprise. Vérifier séparément si une course observable par
un humain reste dans le produit. Ne pas vendre -p=1 comme correction.

Validation et caractère bloquant

Répéter le scénario sous charge et la suite parallèle; aucune fausse
assertion de disponibilité. Diagnostic de course test probable, pas
fuite permanente de verrou établie.

Suivi

En file d’attente issue de la revue, pas un correctif en cours.
Héphaïstos porte le diagnostic et les propositions techniques sur release.
Mathieu arbitre les choix de produit et autorise les fusions.

## Revue de préparation publique Branche de travail : `release`. Code examiné : `6052defc07a8d85a26a62014a72d23c4855a4786`. Sévérité : **medium**. Domaine : **tests**. ## Constat et impact La suite parallèle échoue de façon intermittente au second start du test de livraison bloquée: session-lock, session déjà active. Le même test passe trois fois isolé et la suite complète passe avec -p=1. Le résultat persistant peut être lu avant la fermeture du verrou. ## Preuves internal/acceptance/acceptance_test.go:1418-1424; internal/session/orchestrator.go:188-189; internal/cli/start_unix.go:189,306-309. ## Travail recommandé Synchroniser le test sur la terminaison/libération effective, puis stresser la reprise. Vérifier séparément si une course observable par un humain reste dans le produit. Ne pas vendre -p=1 comme correction. ## Validation et caractère bloquant Répéter le scénario sous charge et la suite parallèle; aucune fausse assertion de disponibilité. Diagnostic de course test probable, pas fuite permanente de verrou établie. ## Suivi En file d’attente issue de la revue, pas un correctif en cours. Héphaïstos porte le diagnostic et les propositions techniques sur `release`. Mathieu arbitre les choix de produit et autorise les fusions.