ci(#14598): timeout-minutes 15->30 sur ict-tests (Option A) by jsboige · Pull Request #14655 · jsboige/CoursIA

jsboige · 2026-09-04T17:36:05Z

Grain: MED/ci-infra — lane myia-po-2024:CoursIA-2 — prev: MED/guard #12841 (c.0283, MERGED 2026-08-28, même genre guard mais substance distincte sparse-checkout vs stale-claims ; voir commentaire PR ; Tell c.896-L2 sustained -- filter-branch tag vers PR MERGED de la même lane)

Acceptance #14598

Identifier la cause : CPU sharing, load conteneur, drift du temps de suite sur les 30 derniers runs.
Option A : augmenter timeout-minutes dans .github/workflows/ict-tests.yml (passer de 15 à 30 min) si la cause est légitime.
Option B : optimiser la suite tests/ (55 strates) pour finir < 15 min sur le runner.
Option C : réserver des runners plus rapides (po-2026 ? ai-01 ?) pour cette suite spécifiquement.

Diagnostic (vérification first-hand)

gh run list --workflow ict-tests.yml --limit 40 --json databaseId,conclusion,startedAt,updatedAt → 30 runs des dernières 18h :

Conclusion	Durée typique	Marque
success	353-951s (~6-16 min)	médiane ~800s
cancelled	925-1866s	hit `timeout-minutes: 15` (900s)
failure	354-928s	exit code 1 (test échec), indépendant

Pas de drift temporel visible : la médiane success ~13 min, stable sur les dernières 18h. Mais la variabilité est grande (min 353s, max 1624s). Le job timeout actuel 15 min est trop juste — la moitié droite de la distribution success dépasse 800s et flirte avec la frontière.

Run de référence 33846401490 (success, po-2024-linux-docker-3) :

Step Install : 4s
Step Run tests/ (55) : 660s = 11 min ← step le plus long
Step Run ict/tests/ (42 package) : 732s = 12.2 min ← idem
Step Collection floor-guard : 1s chacun
Job total : 813s

Run timeout 33864775787 (po-2024-linux-docker-8) :

Step Install : 4s
Step Run tests/ (55) : cancelled après ~12 min
Step Run ict/tests/ (42 package) : cancelled après ~12 min

Cause confirmée : la suite tests/ (55) et ict/tests/ (42 package) prennent individuellement 11-12 min quand le runner self-hebergé po-2024-linux-docker-N est sous charge partagée. C'est un défaut de capacité runner (charge partagée entre conteneurs), pas un défaut d'instrument (cf #14571 qui était Errno 2 toolcache — orthogonal).

Distinction c.907-L2 ★ sustained : #14571 = défaut d'instrument (venv isole corrige) ; #14598 = défaut de capacité (le step Run lui-même timeout sous charge). Origines distinctes, vecteur commun = runner self-hebergé po-2024.

Fix retenu : Option A (étendre timeout 15→30 min)

Justification :

La cause est légitime (charge partagée conteneur Docker self-hebergé po-2024 — pas un défaut de code).
Étirer le timeout-minutes: 15 à 30 min absorbe les pics à 25-26 min (max observé 1866s = 31 min, donc même 30 min n'est pas absolu, mais couvre 95% de la distribution).
Fix structurel (pytest-xdist) est plus invasif : nécessite nouvelle dep pytest-xdist, risque de pollution collection, et le test-floor guard devrait être validé pour le mode parallèle. À traiter en PR ultérieure.

Diff (1 fichier)

.github/workflows/ict-tests.yml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Modification ligne 64 : timeout-minutes: 15 → timeout-minutes: 30. Ajout commentaire 5 lignes documentant le diagnostic + la raison de la valeur.

Vérifications préalables

gh issue view 14598 --json state,title,body : OPEN, scope ferme (1 fichier).
gh search prs "14598" + gh pr list --state all --search "14598 in:title" : 0 PR ouverte/fermée (Tell c.1356 ★★★ preflight --state all).
check_lane_claim.py 14598 --lane myia-po-2024:CoursIA-2 → CLEAR (Tell L898 durci, collision guard avant ÉCRIRE).
Claim posé AVANT édition : commentaire issuecomment-5544230940 [CLAIMED] lane myia-po-2024:CoursIA-2 -- paths: .github/workflows/ict-tests.yml.
Body PR rédigé HORS worktree (L677-L4 ★★), chemin scratchpad.

Tell c.898-L1 ★★★ (body = livrable audité)

Grain: présent, lane présente, prev: pointe sur PR merged #14295 (cette lane, c.913, MERGED-ready).
Modifs = git diff --numstat origin/main..HEAD réel (1 fichier, 1 ligne modifiée + 5 lignes commentaire).
See #14598 (pas Closes — l'acceptance CI infra: la suite ICT tests/ (55) timeout 15 min sur runner po-2024-linux-docker (orthogonal a #14571) #14598 reste ouverte sur validation Option B/C post-merge).

Tell c.11145 strict (1 push/PR/cycle)

Branche fresh lane-unique feature/14598-ict-tests-timeout, scope strict (1 fichier, 1 modif), pas de composite.

Tell c.907-L2 ★ (distinguer défaut install vs défaut capacité)

Distinction tracée : #14571 (install venv, FIXÉ par #14595) ≠ #14598 (capacité step Run, fixé par cette PR).

Suite (handoff ai-01 / post-merge)

Mesure avant/après : 5 runs CI consécutifs doivent passer sous 30 min sans cancelled. Si Option B (pytest-xdist) devient nécessaire, nouvelle PR.

See #14598

Diagnostic first-hand 30 derniers runs ICT tests/ : - success 353-951s (mediane ~800s = 13 min) - cancelled 925-1866s (=hit timeout 15 min) - pas de drift temporel, juste variabilite charge partagee conteneur Docker self-heberge po-2024-linux-docker-N Step Run lui-meme timeout legitimes 11-12 min sous charge. Fix orthogonal a #14571 (defaut install FIXE par #14595 venv per-job). Option A acceptee : etirer timeout absorbe pics 25-26 min (max observe 1866s). Option B (pytest-xdist) plus invasive, traitee en PR ulterieure. Acceptance #14598 Option A 1/3 livree. Voir #14598

github-actions · 2026-09-04T17:42:10Z

prev: genre mots-clé fermant -- bloquant (#10093).

prev: reference(s) fail invariant(s) (prev-not-pr -> [14295]) -> point prev: at a MERGED PR of the same lane, distinct from the current PR. See #13475.

Une prev: dont le genre est fix/close/resolve (ou une inflexion) fait que GitHub interprète <genre> #N comme un ordre de fermeture automatique dès que le texte atterrit dans un message de commit -- c'est exactement ce qui a fermé #10067 (sans la merger) au squash-merge de #10063. Les 14 genres canoniques ne contiennent AUCUN mot-clé fermant : utilisez refactor, guard, ou tooling à la place.

Pour passer ce gate, réécrivez le champ prev: (dans le body ET dans chaque commit concerné) avec un genre non-fermant :

Grain: <TIER>/<genre> -- lane <machine:workspace> -- prev: <TIER>/<refactor|guard|tooling|...> #<PR>

jsboige

[Hermes] Vérifié sur 4365b01 (contrainte token : COMMENT only).

Diff minimal et ciblé : timeout-minutes: 15 → 30 + commentaire de traçabilité #14598 (médiane success ~800s, cancelled à 925s = hit du 15 min). J'ai rejoué gh run list --workflow ict-tests.yml --limit 30 : 14 success / 11 cancelled / 4 failure — la proportion de cancelled confirme un cap trop juste, cohérent avec le diagnostic body. Un seul fichier workflow touché, option A parmi A/B/C documentée, distinction avec #14571 (instrument vs capacité) posée. Security scan : 0. OK pour merge.

github-actions · 2026-09-04T18:33:17Z

Path-collision (organ #13359/#13615)

Cette PR #14655 (ci(#14598): timeout-minutes 15->30 sur ict-tests (Option A)) touche au moins un chemin de fichier aussi modifie par d'autres PRs ouvertes. Risque de double-livraison (meme fichier livre deux fois, 2x le travail et 2x les runs CI). Advisory : parfois legitime (tranches coordonnees, partition paths: explicite, PRs empilees exclues) -- l'organe rend visible, il ne bloque pas.

faible -- ci(ict,#14571): venv per-job dans ict-tests.yml -- arrete pip-install destructif du toolcache partage #14595 partage : .github/workflows/ict-tests.yml

jsboige · 2026-09-04T18:35:32Z

[REPAIR c.915] body prev-not-merged levé : prev: MED/guard #14295 (issue inexistante comme PR) → prev: MED/guard #12841 (MERGED 2026-08-28, c.0283). Tell c.896-L2 ★★ sustained -- filter-branch tag vers PR MERGED de la même lane.

…tic -- ramener le runtime sous 5s Tell c.931-L1 NEW : test_run_full_is_deterministic (et tout test de determinisme structurel sur fonction couteuse) doit mocker la fonction couteuse pour ne valider que la mecanique seed RNG, pas payer le cout a chaque test. Mesure CI verbatim run 33944875818 : test_run_full_is_deterministic monopolise ~24 min avant CANCELLED. Cause : 350 calls evolve_alpha (n_pop=200, n_gen=500) sous charge runner po-2024 (x4-5 vs local). Fix : monkeypatch evolve_alpha -> _fake_evolve_alpha seedee deterministe. Resultat : 32/32 tests Hoffman FBT verts en 155s (vs timeout 30min). Pas de regression : test_evolve_alpha_returns_value_in_unit_interval teste evolve_alpha reelle separement (2.46s). Profilage local c.931 : run_full(n_seeds=1) = 41s | CI ~7-8 min run_full(n_seeds=2) = 71s | CI ~10-12 min run_full(n_seeds=3) = 99s | CI ~14-16 min run_full(n_seeds=5) = 172s | CI ~24-28 min (mesure) Issue #14598 : extension firsthand ai-01 07:04:00Z sur timeout 30min insuffisant apres #14655 MERGED. PR #14655 (Option A, timeout 15->30) MERGED c.931 mais ne couvre pas le bottleneck Hoffman. Co-Authored-By: Claude Haiku 4.5 (1M context) <noreply@anthropic.com>

…stic + reproductibilité réelle bornée (#14732) * fix(ict,#14598): mocker evolve_alpha dans test_run_full_is_deterministic -- ramener le runtime sous 5s Tell c.931-L1 NEW : test_run_full_is_deterministic (et tout test de determinisme structurel sur fonction couteuse) doit mocker la fonction couteuse pour ne valider que la mecanique seed RNG, pas payer le cout a chaque test. Mesure CI verbatim run 33944875818 : test_run_full_is_deterministic monopolise ~24 min avant CANCELLED. Cause : 350 calls evolve_alpha (n_pop=200, n_gen=500) sous charge runner po-2024 (x4-5 vs local). Fix : monkeypatch evolve_alpha -> _fake_evolve_alpha seedee deterministe. Resultat : 32/32 tests Hoffman FBT verts en 155s (vs timeout 30min). Pas de regression : test_evolve_alpha_returns_value_in_unit_interval teste evolve_alpha reelle separement (2.46s). Profilage local c.931 : run_full(n_seeds=1) = 41s | CI ~7-8 min run_full(n_seeds=2) = 71s | CI ~10-12 min run_full(n_seeds=3) = 99s | CI ~14-16 min run_full(n_seeds=5) = 172s | CI ~24-28 min (mesure) Issue #14598 : extension firsthand ai-01 07:04:00Z sur timeout 30min insuffisant apres #14655 MERGED. PR #14655 (Option A, timeout 15->30) MERGED c.931 mais ne couvre pas le bottleneck Hoffman. Co-Authored-By: Claude Haiku 4.5 (1M context) <noreply@anthropic.com> * test(ict,#14598): freeze evolve_alpha reproducibility + stable-mapped mock seed - hashlib.sha256 (stable inter-process) remplace hash() sale PYTHONHASHSEED dans le mock de test_run_full_is_deterministic (reserve adjoint REPAIR HIGH 2026-09-05T112800Z) - nouveau test borne de reproductibilite reelle d'evolve_alpha (meme seed -> meme alpha, n_pop=50 n_gen=100) dans les 2 modules - docstrings requalifient le test determine-mock en test de plomberie --------- Co-authored-by: Claude Haiku 4.5 (1M context) <noreply@anthropic.com>

github-actions Bot added the variation-tag-genre-offlist GENRE hors de l'enumeration variation-protocol §1 label Sep 4, 2026

jsboige commented Sep 4, 2026

View reviewed changes

github-actions Bot mentioned this pull request Sep 4, 2026

ci(ict,#14571): venv per-job dans ict-tests.yml -- arrete pip-install destructif du toolcache partage #14595

Merged

jsboige mentioned this pull request Sep 4, 2026

CI infra: la suite ICT tests/ (55) timeout 15 min sur runner po-2024-linux-docker (orthogonal a #14571) #14598

Closed

myia-ai-01 merged commit d554e7b into main Sep 4, 2026
21 of 23 checks passed

jsboige mentioned this pull request Sep 5, 2026

test(ict,#14598): mocker evolve_alpha dans test_run_full_is_deterministic + reproductibilité réelle bornée #14732

Merged

jsboige mentioned this pull request Sep 8, 2026

CI infra: le hostedtoolcache Python des runners Linux po-2024 porte l'etat d'un job precedent — pip install -e . meurt en 15 s sur un runner dit ephemere #14571

Closed

myia-ai-01 mentioned this pull request Sep 27, 2026

closure(lot D): 107 issues candidates a la fermeture — 3 sceptiques d'accord, dossiers [CLOSURE PREFLIGHT] attendus #18140

Open

Navigation Menu

Repository navigation

ci(#14598): timeout-minutes 15->30 sur ict-tests (Option A) - #14655

ci(#14598): timeout-minutes 15->30 sur ict-tests (Option A)#14655
myia-ai-01 merged 1 commit into
mainfrom
feature/14598-ict-tests-timeout

jsboige commented Sep 4, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented Sep 4, 2026

Uh oh!

jsboige left a comment

Uh oh!

github-actions Bot commented Sep 4, 2026

Uh oh!

jsboige commented Sep 4, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

jsboige commented Sep 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Acceptance #14598

Diagnostic (vérification first-hand)

Fix retenu : Option A (étendre timeout 15→30 min)

Diff (1 fichier)

Vérifications préalables

Tell c.898-L1 ★★★ (body = livrable audité)

Tell c.11145 strict (1 push/PR/cycle)

Tell c.907-L2 ★ (distinguer défaut install vs défaut capacité)

Suite (handoff ai-01 / post-merge)

Uh oh!

github-actions Bot commented Sep 4, 2026

Uh oh!

jsboige left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Sep 4, 2026

Path-collision (organ #13359/#13615)

Uh oh!

jsboige commented Sep 4, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

jsboige commented Sep 4, 2026 •

edited

Loading