Skip to content

ci: track the upstream mbx manifest fix as a pin bump - #6283

Merged
kixelated merged 6 commits into
mainfrom
quest/m0/ci/mbx-r2-verify
Oct 23, 2026
Merged

kixelated merged 6 commits into
mainfrom
quest/m0/ci/mbx-r2-verify

Conversation

@kixelated

@kixelated kixelated commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator

Problem

Hosted R2 cache verification is incomplete. The scheduled macOS platform jobs refuse task action manifests that mbx wrote past its own 16 MiB reader bound, so those tasks get no remote predictions, and no hosted job shows remote reuse yet.

Approach

Upstream report jdx/mr-boxington#792 is closed, fixed by jdx/mr-boxington#798 (commit 9c8d3c7). It caps a task manifest at a 15 MiB serialized budget and keeps the 16 MiB read bound. It merged 27 minutes after v1.27.0 was published, so no release carries it; the open release PR jdx/mr-boxington#797 proposes it as 1.27.1. The pin in .github/actions/rust-cache/action.yml therefore stays at 1.25.0, the hold from #6515.

The quest changes:

  • New quest/m0/ci/mbx-bump.md [XS]: waits for an mbx release containing CPU Maxed Out #798 (checkable with a compare call), then bumps the pin. That release also carries jdx/mr-boxington#789, which ci: hold the Rust cache at mbx 1.25 #6515 named as the upgrade condition for its 1.26 E0463 hold. CPU Maxed Out #798 does not repair manifests already oversized on R2; they stay stuck until the 30-day lifecycle deletes them, about 2026-11-22 at the latest.
  • quest/m0/ci/mbx-r2-verify.md now requires the bump, drops the met bootstrap condition, and records evidence from Nightlies 39600361153, 39779729461 and 39951870847. The 2026-10-23 run on mbx 1.26.0 still refuses manifests, adding a third digest (b0310ce4..., 17228175 bytes).
  • Windows refuses nothing but still reuses nothing: 0 hits, and 1084 of 1091 compilations had no matching prediction to look up, a day after a 1.25.0 Nightly wrote. The manifest fix does not explain this, so the quest says to find why before timing builds if it persists after the bump.

The remaining gates are unchanged and unproven: consecutive hosted remote reuse, a release PR's effective read-only (no release PR has used R2 yet), comparable build timing, and a full month of bucket, request, and expiry data. They wait on the upstream release and on elapsed time. Only scheduled runs were used; nothing was dispatched.

Impact

  • Public API and wire: none. Quest files only.
  • No cache policies, limits, secrets, schedules, or pins change.

Alternatives

Validation

nix develop --command just check passed (quest graph of 574 documents, Markdown, shell, TOML, Nix, justfiles, workflow checks). quest check --base origin/main passed.

Follow-ups

quest/m0/ci/mbx-bump.md is ready as soon as an mbx release contains #798.

(Written by Claude Opus 5.5)

🤖 Generated with Claude Code

@kixelated kixelated changed the title Verify the R2 Rust cache ci: checkpoint hosted R2 cache verification failure Oct 22, 2026
Co-Authored-By: GPT-6 <noreply@openai.com>
@kixelated

Copy link
Copy Markdown
Collaborator Author

Verification checkpoint pushed as 8d55783 after nix develop --command just ci and git diff --check passed. The linked macOS Nightly has effective read-write credentials but repeats the exact 17,204,435-byte task-manifest refusal and reports zero downloaded bytes in all eleven cache summaries. The quest retains the failed verification and the remaining observations. This draft's ## Decisions needed records the upstream-post approval and unfinished hosted reuse/month-retention work. No cache policy, cap, secret, schedule, or dependency pin changed; no external report was sent.

(Written by GPT-6)

Combine main's hosted-jobs decision with this branch's verification
checkpoint into one Plan section.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kixelated

Copy link
Copy Markdown
Collaborator Author

Filed upstream as jdx/mr-boxington#792 (oversized task action manifest: the producer prunes by entry count, the reader caps at 16 MiB, and the stuck object blocks later uploads too). Both observed Nightlies ran mbx 1.25.0; the code involved is unchanged in 1.26.0 and upstream main. The 2026-10-22 Nightly hit a second task the same way (5cb8fb79..., 17213760 bytes).

(Written by Claude Opus 5.5)

kixelated and others added 2 commits October 23, 2026 08:55
jdx/mr-boxington#792 is fixed by jdx/mr-boxington#798, which no release
carries yet. Split the pin bump into its own waiting quest and rewrite the
verification quest to the evidence and the remaining measurements.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kixelated kixelated changed the title ci: checkpoint hosted R2 cache verification failure ci: track the upstream mbx manifest fix as a pin bump Oct 23, 2026
@kixelated
kixelated marked this pull request as ready for review October 23, 2026 16:00
@coderabbitai

coderabbitai Bot commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 16 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 4 included reviews currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 106f10c5-a03b-4908-ae17-c7c1936f0c5f
📥 Commits

Reviewing files that changed from the base of the PR and between 3f72065 and 4e51aed.

📒 Files selected for processing: 3
  • quest/m0/ci/README.md
  • quest/m0/ci/mbx-bump.md
  • quest/m0/ci/mbx-r2-verify.md
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-23T16:03:12.990707Z 0dc40f2 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0dc40f2ee6

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread quest/m0/ci/mbx-r2-verify.md Outdated
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kixelated

Copy link
Copy Markdown
Collaborator Author

Merging. The maintainer approved the split: a waiting quest for the mbx release that carries jdx/mr-boxington#798, with the verify quest rewritten to what remains. Codex reviewed the previous head; the only later commit is the trivial one-line fix for its finding (--event schedule). CI is green, the current head has a non-Grok review, and quest check passes against main.

(Written by Claude Opus 5.5)

@kixelated
kixelated added this pull request to the merge queue Oct 23, 2026
Merged via the queue into main with commit 252e915 Oct 23, 2026
10 checks passed
@kixelated
kixelated deleted the quest/m0/ci/mbx-r2-verify branch October 23, 2026 18:12
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