Skip to content

ci(gametheory): le job scripts-tests ne se declenche pas sur les PR qui modifient GameTheory/** #21886

Closed

Description

@jsboige

Part of #21764 (plancher pyarrow des images CI) — écart relevé par la relecture tierce de la PR #21783, point non bloquant n°2, reporté sciemment avant le merge de #21783.

Constat

Le job scripts-tests porte le step e2e (papermill sur GameTheory-06) ajouté par la PR #21783, mais son déclencheur on.pull_request.paths ne couvre pas la famille :

  • MyIA.AI.Notebooks/GameTheory/** → absent (seuls GameTheory/gateway/**, deux fois) ;
  • MyIA.AI.Notebooks/GameTheory/requirements.txt → absent.

Conséquence mesurée : la PR qui modifie le carnet que le step doit attraper (GameTheory-06, cf. #21662) ne déclenche pas le job. Le garde ne s'exécute que sur les PR suivantes qui touchent scripts/**.

C'est la classe déjà documentée en tête de gametheory-tests.yml (« citation textuelle du déclencheur de paths: »).

Deux remèdes, non appliqués ici

  1. ajouter MyIA.AI.Notebooks/GameTheory/GameTheory-06-*.ipynb (ou GameTheory/**) et GameTheory/requirements.txt aux paths: de scripts-tests.yml ;
  2. déplacer le step e2e dans gametheory-tests.yml, qui déclenche déjà sur GameTheory/**.

Pourquoi ce n'est pas fait dans #21783

Le remède 1 élargit le déclencheur du job le plus sollicité du dépôt (scripts-tests exécute déjà un papermill d'environ 21 s sur toute PR touchant scripts/**) : l'élargissement à GameTheory/** est une décision de coût CI qui relève du coordinateur, pas d'un correctif de fond de la PR. Le remède 2 touche l'organisation des jobs.

Lane : myia-ai-01:CoursIA-2 (traitement de la file de réparation P0). Mesure : déclencheur lu dans .github/workflows/scripts-tests.yml au head 25b481c316.

Activity

  1. jsboige commented on Oct 24, 2026

    @jsboige
    OwnerAuthor

    [CLAIMED] lane myia-ai-01:CoursIA-2 — paths: .github/workflows/scripts-tests.yml -- les paths: du job scripts-tests doivent couvrir ce que le job execute et importe pour la famille GameTheory (tests/**, game_theory_utils.py, requirements.txt)

  2. added 2 commits that reference this issue on Oct 24, 2026
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