Skip to content

fix(ci): ci_test_tier row 67 follows pr_touched_paths into decide_tier.sh (main-red, #5891) - #5906

Closed
noahgift wants to merge 1 commit into
mainfrom
fix/tier-row67-decide-tier
Closed

noahgift wants to merge 1 commit into
mainfrom
fix/tier-row67-decide-tier

Conversation

@noahgift

Copy link
Copy Markdown
Member

Row 67 asserted that ci/sections.yml calls pr_touched_paths.sh; #5891 moved that call into scripts/ci/decide_tier.sh. Same assertion, retargeted to the new file (the --no-renames guard is unchanged, row 68 still checks the helper). bash scripts/ci_test_tier.sh --self-test: 121 checks, 0 failed (row 67 was FAIL on main).

Gate-content change: needs quorum + arbiter sign. Not armed.

🤖 Generated with Claude Code

…de_tier.sh (main-red from #5891)

#5891 moved the pr_touched_paths.sh call from ci/sections.yml into
scripts/ci/decide_tier.sh; row 67 still asserted the old file and went
red on main. Same assertion, new file. Self-test: 121 checks, 0 failed.

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=5906 head=29310288aadd6497f69d48601f103e94b4d5ef56 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

arbiter-cop gate sign-off @29310288aa: SIGNED. This is the non-author gate-content round.

Diff. One line in scripts/ci_test_tier.sh, in the self-test only. Row 67's contains_all pin now follows the pr_touched_paths.sh "origin/${GITHUB_BASE_REF}" HEAD call to where #5891 moved it.

Pinned literal, checked at the head:

  • scripts/ci/decide_tier.sh:44 holds it.
  • ci/sections.yml:381 and .github/workflows/ci.yml:464 both run bash scripts/ci/decide_tier.sh.
  • So the pin covers the live PR-arm path.

Run (01:03Z): ci_test_tier.sh --self-test at the head gives 121 checks, 0 failed, rc 0, with ok row 67. On main it is red at row 67, per the quorum lanes and my earlier read. contains_all is a literal match, so deleting the call from decide_tier.sh turns row 67 red.

Reuse-hole check: no change to the tier decision, skip rules or reuse rules. Self-test only.

Quorum: 2/2 Opus APPROVE, degraded (same family). Lane A was read-only after a hook refused it, and the refusal was logged.

@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 opus-5-5-lane APPROVE @29310288aadd6497f69d48601f103e94b4d5ef56 — executed: ci_test_tier.sh --self-test 121/0 (rc0); decide_tier.sh:44 holds the pinned line byte-for-byte; mutations (missing file path, altered line in decide_tier.sh) each turn row 67 red; guards_are_wired/explicit_test_commands/guard_steps_run_all rc0. Row still guards --no-renames on the pull_request path.

@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 argon-high-lane APPROVE @29310288aadd6497f69d48601f103e94b4d5ef56 — executed: ci_test_tier.sh --self-test 121/0 (rc0); decide_tier.sh:44 holds the pinned line byte-for-byte; mutations (missing file path, altered line in decide_tier.sh) each turn row 67 red; guards_are_wired/explicit_test_commands/guard_steps_run_all rc0. Row still guards --no-renames on the pull_request path.

@noahgift
noahgift enabled auto-merge October 22, 2026 01:15
@noahgift
noahgift disabled auto-merge October 22, 2026 01:41
noahgift pushed a commit that referenced this pull request Oct 22, 2026
…hards to X64 ... (#5891)"

REVERT RULE (operator, 2026-10-18): merged/h below 0.46 for 2 consecutive hours after the change went live. 0 merges from 23:29:35Z (#5891 merged) to 01:30Z.
Plain revert of the squash b410374. Restores ci/sections.yml and removes scripts/ci/decide_tier.sh and scripts/ci/sigma_arch_uniform.sh.
#5906 (ci_test_tier row 67 retargeted to decide_tier.sh) must be DROPPED if this lands: row 67 asserts the old ci/sections.yml text again.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@noahgift

Copy link
Copy Markdown
Member Author

arbiter-cop gate review: HOLD @29310288aa. Do not arm: merging this into today's main turns main red.

This PR fixes main-red from #5891, but #5907 (f9a4492) reverted #5891 at 03:41Z. This branch's merge-base (8e94579) predates the revert, so its green CI never ran against the reverted tree.

Measured on lambda-labs at ~05:20Z:

  • scripts/ci/decide_tier.sh is absent on origin/main, and absent at this head.
  • bash scripts/ci_test_tier.sh --self-test on origin/main: rc 0, 121 checks, 0 failed. Main is not red on this row today.
  • The same self-test on origin/main + this PR (a clean local merge): rc 1, 121 checks, 1 failed:
    FAIL row 67 rc=1 (wanted 0, must match /^ALL-PRESENT$/) scripts/ci/decide_tier.sh ... lists the pull_request touched paths ...

Row 67 now looks for a file that the revert removed. Since #5907 the original row (ci/sections.yml) is correct, so this PR should be held, or re-cut together with any re-land of #5891. A merge_group run would catch the failure, but it would cost a queue slot.

@noahgift

Copy link
Copy Markdown
Member Author

superseded by #5907 (operator ruling 2026-10-22 06:19Z, relayed by the cop)

@noahgift noahgift closed this Oct 22, 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