Repository navigation
fix(picker): la classe infra_rerun suppose « vert sur main » sans lire la conclusion du job sur la base #19767
Description
Activity
[CLAIMED] lane myia-ai-01:CoursIA-2 — correctif d'organe du picker (posé par le coordinateur au dispatch)
- added a commit that references this issue
on Oct 8, 2026 [INFO] candidate-delivered — lane myia-ai-01:CoursIA-2
Le correctif de cette issue est deja sur
main, livre par la PR #19769 (merged=true, 2026-10-08T10:47:00Z, merge commita3a4d09d3, branchefix/19767-picker-infra-rerun-base-read). L'issue est restee ouverte parce que le corps de la PR portaitSee #19767, pasCloses. Aucune reimplementation : je verifie l'artefact surmaincritere par critere.Ancrage.
git log origin/main --grep 19767->a3a4d09d3 fix(picker,#19767): la classe infra_rerun lit le dernier run push de main, pas le rollup (#19769).Critere du body Etat Preuve sur mainLa classe infra_rerunn'est attribuee que si la conclusion du meme job sur la base est lue vertecouvert scripts/pick_idle_grain.py, dansfetch_main_head_probe: un motif non-Nonede_main_red_motifajoute auxred_keysles noms de check du mapping_WORKFLOW_YML_TO_CHECK_NAMES— le check passe donc enbase_inheritedparsplit_base_corroboration, pas eninfra_rerunRouge sur main->base_inheritedcouvert meme mecanisme ; le probe enrichi porte la corroboration de base Illisible -> base_undecidedcouvert par le repli fail-CLOSED quand _main_red_motifrendNone(vert ou illisible) le probe est rendu inchange : on ne touche pas a la classification, on retombe sur le comportement d'avant #19767. Un import qui echoue ou une exception rendent aussi le probe inchangeControle positif : rouge sur mainne produit jamaisinfra_reruncouvert scripts/tests/test_pick_idle_grain.pyl.855Controle negatif : vert sur mainproduit bieninfra_reruncouvert scripts/tests/test_pick_idle_grain.pyl.874Reutiliser merge_dwell._main_red_motif(organe-first), pas un second lecteurcouvert from ci.merge_dwell import _main_red_motif, MAIN_RED_WORKFLOWSdansfetch_main_head_probeTrois tests de bord en plus :
_main_red_motifqui leve -> probe inchange (l.888) ;probe=None(rollup vide) ->None(l.903) ; motif rouge pour un workflow horsMAIN_RED_WORKFLOWS-> aucune propagation silencieuse (l.919) ; plus le test d'integration de l'appel (l.933).Nuance declaree. Sur le troisieme critere, l'implementation ne force pas litteralement
base_undecidedquand la lecture est illisible : elle rend le probe inchange (fail-CLOSED), et c'estsplit_base_corroborationqui tranche ensuite. L'intention du critere — ne jamais elargirinfra_rerunsans preuve — est tenue ; la lettre (« illisible ->base_undecided») ne l'est pas au sens strict. Je le signale plutot que de le passer sous silence.La fermeture revient au coordinateur ou a l'adjoint (#15069) : je ne ferme pas d'issue moi-meme.
Constat (mesure de la lane
myia-po-2026:CoursIA, 07/10 ~16:00Z)pick_idle_grain.pya classé le rougeScripts Tests (CPU)en « INFRA D'EXECUTION -- rouge ici, VERT sur main : le geste est le REJEU », corroboré par 4 PRs (#19338, #19705, #19708, #19719).La prémisse « vert sur main » était fausse : à ce moment,
mainéchouait sur ce même workflow, sur troispushconsécutifs (14:24:35Za32a8528, 14:32:43Zfadbbc01, 14:45:14Zab6aa5b2), à cause du doublon d'index0019du registre jumeau, que #19723 a corrigé. Le module DWELL le voyait :merge_dwell._main_red_motif("jsboige/CoursIA")rendaitmain rouge.Une lane qui suit l'organe rejoue sa jambe, retrouve le même rouge, et conclut à une instabilité qui n'existe pas. Le rouge revient au tour suivant.
Attendu
infra_rerunn'est attribuée que si la conclusion du même job sur la base (dernier runpushdemainpour ce workflow) est lue verte. Si elle est rouge, le rouge va enbase_inherited; si elle est illisible, enbase_undecided.mainne doit jamais produireinfra_rerun.mainproduit toujoursinfra_rerun.merge_dwell._main_red_motifplutôt qu'en écrire une seconde (organe-first).See #17154 (origine de la classe
infra_rerun).