Skip to content

Commit 43032bb

Browse files
jsboigeclaudemyia-ai-01
authored
ci(#14295): convertir 3 gardes à sparse-checkout (tranche 1) — 91s → ~6s par garde (#18853)
* ci(#14295): convertir 3 gardes a sparse-checkout (tranche 1) — 91s -> ~6s par garde Trois gardes ne lisent que .github/scripts/ (zero corpus MyIA.AI.Notebooks) : - dryrun-sizing-control : scripts/ci/docker/linux-runner/persist/ai-01/ - local-path-waiver-guard : scripts/check_local_path_waivers.py - twin-attestation-name-guard : scripts/notebook_tools/twin_pairs.d/ Conversion chirurgicale sparse-checkout uniquement, aucun changement de logique de garde. local-path-waiver-guard recoit aussi ses paths: triggers manquants (self-cover #8822, meme pattern que dryrun-sizing-control). Controle positif valide : dryrun-sizing-control abandonne rc=2 (garde rouge) quand le script sous test est retire d'un checkout sparse simule (git clone --filter=blob:none + sparse-checkout set). Les 2 autres gardes sont des gardes Python/simples dont le fichier sous test est couvert par leur sparse-checkout (verifiable a la prochaine execution CI). Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> * ci(#18853): retirer le filtre paths: du garde local-path-waiver Review ai-01 5395319838 : le bloc `paths:` supprimait le run `pull_request` sur presque toutes les PRs, or c'est la seule surface ou l'avertissement s'ecrit sur la PR -- le run `issue_comment` s'attache a `main`, jamais a la tete de PR. Le sparse-checkout suffit au gain de temps ; le filtre retirait la visibilite du verdict sans rien ajouter au budget. Aucun garde casse : `check_workflow_label_paths.py` (#8822) ne cible que les workflows a la fois paths-filtres ET poseurs de label, et ce garde fait `--report-only` (aucun label pose). Verifie localement apres retrait : 0 violation, exit 0. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 5.5 <noreply@anthropic.com> Co-authored-by: myia-ai-01 <myia.ai.01.myia@gmail.com>
1 parent f90492d commit 43032bb

3 files changed

Lines changed: 18 additions & 0 deletions

File tree

‎.github/workflows/dryrun-sizing-control.yml‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,10 @@ jobs:
6161
steps:
6262
- name: Checkout PR
6363
uses: actions/checkout@v4
64+
with:
65+
sparse-checkout: |
66+
.github/workflows/dryrun-sizing-control.yml
67+
scripts/ci/docker/linux-runner/persist/ai-01/
6468
6569
- name: Syntaxe des deux scripts
6670
run: |

‎.github/workflows/local-path-waiver-guard.yml‎

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,12 @@ name: Local path waiver guard
1010
on:
1111
issue_comment:
1212
types: [created, edited]
13+
# Pas de filtre `paths:` (#18853, review ai-01 5395319838) : il supprimait le
14+
# run `pull_request` sur presque toutes les PRs, or c'est la SEULE surface ou
15+
# l'avertissement s'affiche sur la PR -- le run `issue_comment` s'attache a
16+
# `main`, jamais a la tete de PR. Un garde dont le verdict doit rester visible
17+
# sur la PR ne peut pas etre filtre par chemins ; le sparse-checkout suffit au
18+
# gain de temps (91 s -> ~6 s).
1319
pull_request:
1420
types: [opened, synchronize, reopened]
1521
workflow_dispatch:
@@ -42,6 +48,10 @@ jobs:
4248
runs-on: ubuntu-latest
4349
steps:
4450
- uses: actions/checkout@v4
51+
with:
52+
sparse-checkout: |
53+
.github/workflows/local-path-waiver-guard.yml
54+
scripts/check_local_path_waivers.py
4555
- uses: actions/setup-python@v5
4656
with:
4757
python-version: '3.11'

‎.github/workflows/twin-attestation-name-guard.yml‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,10 @@ jobs:
5050
steps:
5151
- name: Checkout
5252
uses: actions/checkout@v4
53+
with:
54+
sparse-checkout: |
55+
.github/workflows/twin-attestation-name-guard.yml
56+
scripts/notebook_tools/twin_pairs.d/
5357
5458
- name: Gate twin_pairs.d component lengths
5559
run: |

0 commit comments

Comments
 (0)