docs(ci): record completed Depot transition evidence - #1125
Merged
Merged
Conversation
Contributor
📝 WalkthroughWalkthroughCI documentation now records completed packaging and runner-image migrations, successful validation runs, current Depot canary evidence, and the disabled global rollout gate pending administrative trust-boundary verification. ChangesCI migration evidence
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@ci/ci.md`:
- Around line 574-587: Update the “Cold/warm six-label canaries” paragraph so it
lists the allowed canaries as passed, but describes the denied feature-ref
canary as “skipped as expected.” Keep the existing details about the exhaustive
prerelease, warm non-GPU release canary, and feature-ref cleanup unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: e00c305e-e9b5-4a35-adef-ca098a56c696
📒 Files selected for processing (3)
.agents/skills/manage-ci/references/current-inventory.mdci/DEPOT_MIGRATION.mdci/ci.md
Comment on lines
+574
to
+587
| Cold/warm six-label canaries | ||
| [30525111329](https://github.com/Mesh-LLM/mesh-llm/actions/runs/30525111329) | ||
| and | ||
| [30525247727](https://github.com/Mesh-LLM/mesh-llm/actions/runs/30525247727), | ||
| denied feature-ref canary | ||
| [30593657371](https://github.com/Mesh-LLM/mesh-llm/actions/runs/30593657371), | ||
| exhaustive prerelease | ||
| [30586470043](https://github.com/Mesh-LLM/mesh-llm/actions/runs/30586470043), | ||
| and warm non-GPU release canary | ||
| [30590595090](https://github.com/Mesh-LLM/mesh-llm/actions/runs/30590595090) | ||
| all passed. The warm release canary used nine Depot jobs, restored both static | ||
| ABI inputs without compilation, and produced roughly 95% sccache hit rates in | ||
| both Linux native-SDK consumers. The feature-ref run was skipped before runner | ||
| allocation and its main-identical temporary ref was removed. |
Contributor
There was a problem hiding this comment.
🔒 Security & Privacy | 🟠 Major | ⚡ Quick win
Do not report the denied feature-ref canary as passed.
The same paragraph says all canaries passed, but the feature-ref run was skipped before runner allocation. Describe the allowed canaries as passed and the denied feature-ref canary as “skipped as expected,” matching ci/DEPOT_MIGRATION.md Lines [298-301] and current-inventory.md Lines [278-281].
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@ci/ci.md` around lines 574 - 587, Update the “Cold/warm six-label canaries”
paragraph so it lists the allowed canaries as passed, but describes the denied
feature-ref canary as “skipped as expected.” Keep the existing details about the
exhaustive prerelease, warm non-GPU release canary, and feature-ref cleanup
unchanged.
This was referenced Aug 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
v0.75.0-rc1release and warm non-GPU cache evidence;DEPOT_RUNNERS_ENABLEDunset untilmainhas an enforceable review/status boundary and the runner-group allowlist is verified.Evidence
Validation
git diff --check;main(851888d0b0ce19916d6b0d7d73ce49246eef67d6).Summary by CodeRabbit