Skip to content

work/5950-v16: affected_set, ladder, LAB workflow, merge-group fixes (7 commits) - #6007

Closed
noahgift wants to merge 7 commits into
mainfrom
work/5950-v16
Closed

noahgift wants to merge 7 commits into
mainfrom
work/5950-v16

Conversation

@noahgift

Copy link
Copy Markdown
Member

work/5950-v16: affected_set, ladder, LAB workflow and merge-group fixes (7 commits)

Linear on main, no merge commits, one PMAT-Ticket per commit (work/5950-v16 @b0959ada9d). Supersedes #6006 (v15) and #5991 (v13) and the v12 branch. v16: the affected_set --self-test left x86-main and is now a guards-nightly LAB step with a lab_ticket opener and manifest row (wired check: 3 unwired, ratchet PASS); merge-group timeouts 100->120, above the shard's inner 110-min step. Built by w5032. Not armed; seats pending.

ont-delta: none no new type, shape or reason; fold of existing tickets

🤖 Generated with Claude Code

noahgift and others added 7 commits October 22, 2026 23:16
…LAB nightly schedules (PMAT-5847)

Refs #5847

PMAT-Ticket: PMAT-5847
PMAT-Batch: BATCH-aprender-build-speed
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…a so a REBASE-replayed PR is judged whole

Refs #5859

PMAT-Ticket: PMAT-5859
PMAT-Batch: BATCH-aprender-build-speed
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…IGKILLed

Refs #5868

PMAT-Ticket: PMAT-5868
PMAT-Batch: BATCH-aprender-build-speed
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…+3 ticks

Refs #5798

PMAT-Ticket: PMAT-5798
PMAT-Batch: BATCH-aprender-build-speed
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…40->120 / 180->120; shard permissions block added; LAB audits created

x86-main and workspace-test-shard each gain a failure() && merge_group step that runs
`gh run cancel`. merge_group timeouts drop to 120 min (x86-main was 240, the shard 180), above the shard's inner 110-min nextest step (ci/sections.yml) so the step budget fires first;
pull_request/push keep 240/180. workspace-test-shard gets a NEW permissions block
(contents: read, checks: read, actions: write) so the cancel is authorised and the
check-suite reader keeps working. Adds ci/step-budgets.tsv and the LAB tools
scripts/lab/ci_failfast_audit.sh and scripts/ci/step_budget_check.sh, which no PR or
release job runs (C324 r5).

The cancel is a request: whether it beats the always() gate is unmeasured (see
check_ci_unwedge.sh:6-24, #3229) and pending quorum.

Refs #5394

PMAT-Ticket: PMAT-5394
PMAT-Batch: BATCH-aprender-build-speed
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…s.sha before checkout

The kfocus entry in check_runner_labels.sh DISCRIM was put to a quorum round
(Opus+Haiku, bound to 1495fe5): AGREED 2/2, not a weakening.

Refs #5889

PMAT-Ticket: PMAT-5889
PMAT-Batch: BATCH-aprender-build-speed
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…eck that does not exist

scripts/affected_set.sh cited scripts/check_affected_readers.sh, which is not in the tree. The
comment now says what is true (a crate missing from READERS is skipped silently). A 12-row
--self-test on a synthetic graph covers the empty diff, closure + readers, a leaf, a CORE crate,
non-.rs under crates/, build.rs, an unplaced path, docs, ALL winning over a placed path and both
--filter forms. Dropping the build.rs case turns its row RED.

Refs #5974

PMAT-Ticket: PMAT-5974
PMAT-Batch: BATCH-aprender-silicon-coverage
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

@noahgift noahgift left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seat 1 (aprender-w5404, non-author): APPROVE @b0959ada9da85ffe33cb5fb7c1ebac405f29a13c. EMG-7 GREEN (7 commits, one PMAT-Ticket each, 0 merges, base a743e8c). Delta vs v15: affected_set --self-test moved from x86-main to a guards-nightly LAB step (opener + manifest row; check_guards_are_wired.sh rc 0; self-test 12 rows ok), so no new GATE; merge_group timeouts 120 (>= inner 110-min step), still half the old 240. Privacy scan clean. Timeout change is a two-way door (revert rule applies). Earlier scripts-half review (v15) stands.

@noahgift

Copy link
Copy Markdown
Member Author

Snapshot: head b0959ad. Superseded by newer open #6010 on the same topic (PR-explosion cleanup, cop order aprender-57; EMG-7 tickets verified on main or in the successor). Branch kept; closing.

@noahgift noahgift closed this Oct 23, 2026
@noahgift noahgift reopened this Oct 23, 2026
@noahgift

Copy link
Copy Markdown
Member Author

superseded by #6010; branch kept. (dead-PR sweep, operator 08:46Z via aprender-b9)

@noahgift noahgift closed this Oct 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant