Skip to content

fix(ci): pr-light runs on an X64 clean-room runner (Refs #6239) - #6240

Closed
noahgift wants to merge 1 commit into
mainfrom
fix/pr-light-runner-label
Closed

noahgift wants to merge 1 commit into
mainfrom
fix/pr-light-runner-label

Conversation

@noahgift

@noahgift noahgift commented Oct 24, 2026 •

Copy link
Copy Markdown
Member

Closes #6239. GATE CHANGE.

  • ci.yml pr-light runs-on -> [self-hosted, Linux, X64, clean-room] (as CI: pin ci/gate, gate and mg-reuse-self-test to X64 runners #6200 did for the gate job); scripts/ci_mg_reuse.sh pins that line and is updated with it.
  • That script also had a guard forbidding any X64 label on pr-light. It now forbids only the merge-queue label (mq); X64 is allowed. ci_mg_reuse.sh --self-test: 180 passed, 0 failed.
  • X64 clean-room runners online when checked (one gh api call): framework16, framework16-2, framework16-3, kfocus, kfocus-2; all idle.
  • check_runner_labels.sh untouched; clean-room kept.
  • One-line revert: restore the pr-light runs-on line, the pin in ci_mg_reuse.sh, and the guard regex.

Unarmed: needs quorum, two non-author seats, sign.

🤖 Generated with Claude Code

…d host (Refs #6239)

GATE CHANGE. pr-light runs-on moves from [self-hosted, Linux, clean-room]
to [self-hosted, Linux, X64, clean-room], as #6200 did for the gate job.
scripts/ci_mg_reuse.sh pins that line and its heavy-label guard now
forbids only the merge-queue label; X64 is allowed (the clean-room X64
pool). Revert: restore the two lines and the guard regex. The guard still
reds on a pr-light that carries mq.

PMAT-Ticket: PMAT-6239
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

§13.11 rung 1 — quorum shadow verdict

S13-SHADOW pr=6240 head=5912e683ac88405aa9d5e1764ea025a796badf0e verdict=REFUSE class=Q1 arm_rc=1

Shadow mode: this records a verdict and merges nothing. A refusal
to arm is not a block (§13 adds zero rows to §7) — the pull request is
exactly as green as it was.

@noahgift

Copy link
Copy Markdown
Member Author

quorum-review (AD-04): three PASS — agreed (auto_merge: checked=true was_armed=false disarmed=false)

{
 "ticket": "PMAT-6239",
 "head": "5912e683ac88405aa9d5e1764ea025a796badf0e",
 "width": 3,
 "executor": "agy",
 "agreed": true,
 "auto_merge": {
  "checked": true,
  "was_armed": false,
  "disarmed": false,
  "note": "auto-merge not armed"
 },
 "lanes": [
  {
   "lane": 1,
   "verdict": "PASS",
   "findings": 4
  },
  {
   "lane": 2,
   "verdict": "PASS",
   "findings": 5
  },
  {
   "lane": 3,
   "verdict": "PASS",
   "findings": 0
  }
 ]
}

@noahgift

Copy link
Copy Markdown
Member Author

quorum-review (AD-04): NOT agreed (auto_merge: checked=true was_armed=false disarmed=false)

{
 "ticket": "PMAT-1955",
 "head": "5912e683ac88405aa9d5e1764ea025a796badf0e",
 "width": 3,
 "executor": "agy",
 "agreed": false,
 "auto_merge": {
  "checked": true,
  "was_armed": false,
  "disarmed": false,
  "note": "auto-merge not armed"
 },
 "lanes": [
  {
   "lane": 1,
   "verdict": "PASS",
   "findings": 0
  },
  {
   "lane": 2,
   "verdict": "FAIL",
   "findings": 4
  },
  {
   "lane": 3,
   "verdict": "PASS",
   "findings": 0
  }
 ]
}

@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.

PASS 5912e68 (non-author 2nd seat, gate change). patch-id 940bbe9df10d 5912e68. Read .github/workflows/ci.yml and scripts/ci_mg_reuse.sh: pr-light runs-on becomes [self-hosted, Linux, X64, clean-room] (comment updated), the exact-label pin at ci_mg_reuse.sh:603 moves with it, and the heavy-label guard at :623 narrows from X64|mq to mq only. Measured at this head: ci_mg_reuse.sh --self-test 180 passed, 0 failed. Mutant: putting [self-hosted, Linux, X64, mq] on pr-light turns the self-test red (FAIL pr-light: runs-on carries a merge-queue label; 178 passed, 2 failed), so the narrowed guard and the pin both still bite. mg-reuse-wt and the mq lane rows are untouched. The loosening is exactly the stated gate change (X64 allowed, mq still forbidden); no waiver, no third state. Seat counts once pr-light is green at this head.

@noahgift

Copy link
Copy Markdown
Member Author

quorum-review (AD-04): three PASS — agreed (auto_merge: checked=true was_armed=false disarmed=false)

{
 "ticket": "PMAT-1955",
 "head": "5912e683ac88405aa9d5e1764ea025a796badf0e",
 "width": 3,
 "executor": "agy",
 "agreed": true,
 "auto_merge": {
  "checked": true,
  "was_armed": false,
  "disarmed": false,
  "note": "auto-merge not armed"
 },
 "lanes": [
  {
   "lane": 1,
   "verdict": "PASS",
   "findings": 0
  },
  {
   "lane": 2,
   "verdict": "PASS",
   "findings": 3
  },
  {
   "lane": 3,
   "verdict": "PASS",
   "findings": 0
  }
 ]
}

@noahgift

Copy link
Copy Markdown
Member Author

SEAT (non-author, aprender-w5032) bound to 5912e68. patch-id 940bbe9df10d5014e0c26b6573a00f44b6b12ef2. pr-light green at this head: run 40064412878 attempt 2, runner yoga-build (attempt 1 was intel-clean-room-11, red on R-2 body only). Content: pr-light runs-on [self-hosted, Linux, X64, clean-room]; ci_mg_reuse.sh wire row pins the exact line, mq still forbidden; self-test 180/0; fragment guard PASS. Note: mergeStateStatus DIRTY — needs author re-cut/merge of main before arming.

🤖 Generated with Claude Code

@noahgift

Copy link
Copy Markdown
Member Author

Closed by the aprender cop: superseded by merged #6243 (same change). The branch is kept, so reopen if anything here is not in #6243.

@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.

ci: pr-light runs on an X64 clean-room runner, not gx10-build

1 participant