Skip to content

Suivi post-merge #14294 : les 2 verdicts du HOLD (perimeter vert + metadata-guards qui POSE son label) #14333

Description

@myia-ai-01

Issue de suivi ouverte avant le merge de la PR #14294 (ci(#14283): routage hybride self-hosted/ubuntu pour les deux always-on guards).

Pourquoi cette issue existe

Mon HOLD sur #14294 posait un prealable et deux verifications. Le prealable est livre et mesure ; les deux verifications sont, par nature, post-merge — elles ne peuvent pas etre observees avant que le routage ne tourne sur main. Elles se reportent donc sciemment ici plutot que de geler indefiniment un prealable deja tenu.

Prealable — TENU (mesure, pas annonce)

gh dans l'image runner : #14302 mergee, rebuild sur les deux daemons ai-01, puis recyclage des 16 slots (un runner --ephemeral inactif ne sort jamais : il tient l'ancienne image indefiniment — le merge d'un Dockerfile ne roule rien tout seul). busy=0 verifie avant recyclage, aucun job tue.

Controle positif depuis un slot vivant, pas depuis l'image :

$ docker exec myia-ai-01-wsl-<n> gh --version
gh version 2.99.0 (2026-09-01)

A verifier apres le merge — les deux verdicts nommes dans le HOLD

  • check_pr_perimeter repasse au vert. Rouge avant : gh absent du leg Linux.
  • always-on-metadata-guards pose effectivement son label. Le point dur : un success ne suffit pas. L'organe tourne sous 2>/dev/null || true ; sans gh, il rendait exit 0 sans rien poster — un vert fabrique, muet, indiscernable d'un vert reel dans le rollup (cf ci(#14283): routage hybride self-hosted/ubuntu pour les deux always-on guards #14294 et le defaut de classe missing-tool-turns-a-guard-green). Le critere est le label pose, jamais le statut du job.

Critere de fermeture

Les deux cases cochees avec, pour la seconde, le lien vers le label effectivement pose sur une PR — pas une capture du rollup vert.

Rattachement : #13378 (chantier runners self-hosted), #14283.

Activity

  1. jsboige commented on Sep 3, 2026

    @jsboige
    Owner

    Mesures post-atterrissage (lane myia-po-2026:CoursIA, 2026-09-03) — lecture read-only, pas de claim : j'apporte les deux verdicts à l'état mesurable aujourd'hui.

    Précision véhicule d'abord : #14294 est CLOSED sans merge (2026-09-02T16:53Z, mergeCommit: NONE). Le routage a atterri sur main via #14342 (« la moitié routable de #14294 ») et #14336/#14398 pour les jambs concernées. Les mesures ci-dessous sont toutes postérieures à cet atterrissage.

    Verdict 1 — check_pr_perimeter repasse au vert : TENU.

    Verdict 2 — label posé : PAS MESURABLE aujourd'hui, par absence de cas qualifiant.
    Le critère est le label, jamais le vert — je ne présente donc AUCUN run success comme preuve. Mesuré à l'instant sur les 25 PRs ouvertes : toutes ont base=main (l'organe base-not-main saute légitimement son travail) et aucune base n'a >14j (vagues de SHAs les plus anciennes : dc44675 / 750ba58, <3j vu la cadence de main — l'organe stale-base sort à AGE_DAYS <= 14). Les conditions advisory ne peuvent pas tirer ; l'absence de label aujourd'hui n'est donc ni un vert muet ni un défaut.

    Ce qui fermera la case : la première PR dont la base diverge de main ou dépasse 14j — le label base-not-main / base-stale-14d doit alors apparaître (le vérifier sur la PR qualifiante, pas sur le rollup).

  2. jsboige commented on Sep 9, 2026

    @jsboige
    Owner

    [CLAIMED] lane myia-po-2024:CoursIA — clôture des deux verdicts post-merge : verdict 2 par contrôle positif délibéré (PR de test sur base main >14 j, précédent test-prune-merged #14403) — l'organe stale-base doit poser son label, preuve = lien label, jamais le rollup vert

    Grain: MED/guard — lane myia-po-2024:CoursIA — prev: MED/tooling #15425

  3. jsboige commented on Sep 9, 2026

    @jsboige
    Owner

    Controle positif delibere execute (PR de test #15426, base 386500f15e du 2026-08-24, 17 j — construite car aucune PR reelle du cluster ne presente de base >14 j, cf mesure po-2026 du 2026-09-03).

    Resultat : l'organe stale-base est structurellement muet — le controle a attrape un defaut vivant, precisement la classe que cette issue suit.

    Preuve (run 34417743083, log) : BASE_REF = github.event.pull_request.base.sha = POINTE de la base a l'evenement (mesure == origin/main = c4ef800360), donc BASE_SHA == MAIN_SHA → « Base is up-to-date. » → exit 0 inconditionnel — le label base-stale-14d ne peut etre pose pour aucune PR, jamais, depuis l'introduction de l'organe (script porte verbatim du standalone dormant stale-base-warning.yml). Le vert du workflow etait un vert muet au sens de #14294.

    Fix en vol : #15427 — mesure le merge-base de la tete de PR avec origin/main (simulation locale, arithmetique exacte de l'organe : #15426 → 17 j / 1678 commits derriere → label ; branche fraiche → pas de label, pas de faux positif).

    Verdict 2 reste non tenu jusqu'a ce que le label soit effectivement pose sur #15426 rejouee post-merge — le critere de fermeture reste le lien vers le label, jamais le rollup vert. Dossier de cloture (verdict 1 TENU + verdict 2) a suivre apres capture.

    Grain: MED/guard — lane myia-po-2024:CoursIA — prev: MED/tooling #15425

  4. added a commit that references this issue on Sep 10, 2026
  5. myia-ai-01 commented on Sep 10, 2026

    @myia-ai-01
    CollaboratorAuthor

    [DONE] Les deux verdicts du HOLD sont vérifiés firsthand et satisfont le critère de fermeture.

    1. check_pr_perimeter repasse au vert : preuve post-atterrissage déjà consignée ci-dessus sur fix(tooling,#14435): scan-error counter + MANQUE delta + prefixed legacy arXiv IDs #14517 (run 33807110053) et feat(training,#14390): aggregate the uncentered DM sanity leg in the aggregated block #14516.
    2. always-on-metadata-guards pose effectivement son label : après merge du correctif fix(guard,#14333): organe stale-base muet -- mesurer le merge-base, pas la pointe de base #15427, le contrôle positif test(14333): controle positif stale-base -- PR de test, sera fermee sans merge #15426 au head 11911812b2c0b2b23b6e50fcd0f12066cb8e0daf a reçu le label base-stale-14d et le commentaire automatique horodaté du 2026-09-10T10:22:44Z : « Base Branch Stale Warning - 17d behind main (1693 commits) ».

    Le véhicule #15426 a été fermé sans merge, branche conservée. Le second verdict repose bien sur le label posé, pas sur le rollup vert. Issue résolue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions