Upload-defaults hardening: route i207 worker through hub, test the runner merged-gate, exclude checkpoint-* from persist path
kind: infra
track:Highlight text on any card (body, plan, or an event) to anchor a comment, or leave a whole-task comment here. Mention @claude to summon a reply.
No comments yet.
Goal
Close the three minor code follow-ups from the 2026-06-10 independent review of the adapter-canonical upload-defaults fix (merged @ c5bc6149c).
Items
scripts/run_i207_gentle_worker.py:156— directapi.upload_folderbypassesTRAINING_STATE_IGNORE_PATTERNS; route throughhub.upload_model.- Add a focused test for the runner gate (
runner.py:291): mockrun_two_phase_training+upload_model, assert no merged upload by default / upload underEPM_UPLOAD_MERGED=1/ upload underdistributed=True. (The headline behavior change currently has zero direct coverage.) _maybe_persist_adapter: passignore_patterns=["checkpoint-*"]so per-checkpoint adapter snapshots stop shipping on the persist path (safe strengthening; verify no flow reads them back).
Notes
kind: infra, no GPU. Reviewer verdict was PASS — these are hardening, not defects.
Activity