Repository navigation
Conversation
… the merge path to guards-nightly (PMAT-5873) Moved, not deleted: 817 leaves ci/explicit-test-commands.d and runs nightly. Refs #5873 Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
noahgift
left a comment
There was a problem hiding this comment.
Seat (aprender-triage-worker, Sonnet) @2c47dc47f6: REQUEST-CHANGES, one blocker. scripts/check_roadmap_fragment_required.sh exits 1 on this head: 'DRIFT: docs/roadmaps/roadmap.yaml is NOT aggregate(entries/)'. The PMAT-5873 fragment was added without make roadmap-aggregate; guard-tree runs that guard, so the required job will go red. Fix: run make roadmap-aggregate and commit docs/roadmaps/roadmap.yaml. The GATE->LAB move itself is clean (817 removed, guard-tree skips the guard as wired-elsewhere, no continue-on-error, nightly is schedule/dispatch only; manifest/explicit-test/wired/isolated guards rc 0). Non-blocking: add timeout-minutes (~15) to the nightly step, and consider id + if !cancelled() + a lab_ticket opener so a red does not skip later steps. Quorum: Opus (executed) RC, agy APPROVE (missed the drift). I will re-seat on the new head. Receipt handoff/triage-quorum-5874-2c47dc47f6.md. Not armed.
noahgift
left a comment
There was a problem hiding this comment.
Seat (triage-worker, Sonnet 5.5) REQUEST-CHANGES @2c47dc47f6. Quorum AGREED 3/3 REQUEST-CHANGES on one mechanical blocker (Opus executed, agy, me): docs/roadmaps/roadmap.yaml not regenerated, check_roadmap_fragment_required rc=1 on the PR guard path. Fix = make roadmap-aggregate + commit roadmap.yaml (Opus verified the diff is exactly the 20-line PMAT-5873 entry). The GATE->LAB move is clean 3/3 (only invocation left is guards-nightly.yml, 817 deleted, guard_tree skips it, no continue-on-error, schedule-only). Non-blocking: no lab_ticket opener, add if: !cancelled() or move step last. I will re-seat APPROVE when only roadmap.yaml changes. Receipt: handoff/triage-quorum-5874-2c47dc47f6.md
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
noahgift
left a comment
There was a problem hiding this comment.
Seat triage-worker APPROVE @6040e299b4. quorum 3/3 AGREED APPROVE (carried from the 2c47dc4 round; delta = roadmap.yaml +20 only, check_roadmap_fragment_required rc=0 executed on this head). Receipt: handoff/triage-quorum-5874-6040e299b4.md. Non-blocking: lab ticket opener, if: !cancelled().
noahgift
left a comment
There was a problem hiding this comment.
Seat 2 (aprender-w5032, non-author): APPROVE @ 6040e29. No w5032 trailer on either commit. Read the diff: the 817 fragment is removed from ci/explicit-test-commands.d, and the same check is added to guards-nightly.yml and scripts/guards_nightly_manifest.txt under one identical step name. check_guards_nightly_manifest.sh at this head: rc 0, 14 manifest entries, every one a step in the workflow. The PMAT-5873 entry and roadmap.yaml carry the same 20 lines. No third state on the PR path: the check simply leaves it. Not run: the nightly itself, and the second PR that fixes the apr-stub race.
|
Gate-accept sign-off (arbiter-cop, non-author): APPROVE @ 6040e29. This moves check_harness_server_cleanup.sh from ci/explicit-test-commands.d into guards-nightly (LAB), as containment for the #5873 stub-server race. The root cause is runner thread overload; that fix is tracked separately. The quorum's earlier blocker was the roadmap regeneration. The delta at this head is roadmap.yaml only, check_roadmap_fragment_required gives rc=0, and |
|
§13.11 rung 1 — quorum shadow verdict Shadow mode: this records a verdict and merges nothing. A refusal |
|
superseded by #5927 (EMG-001 epic build-speed; this ticket's commit is in its range) |
Refs #5873. Noah 15:04Z item 4 (verbatim via arbiter-cop): "apr stub" race: quarantine the test to nightly LAB, then fix. Never delete.
🤖 Generated with Claude Code