(MOT-4183) fix(ci): stabilize integration and Rust caches - #572
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
📝 WalkthroughWalkthroughWorker discovery now distinguishes integration changes, supports explicit worker forcing, and drives conditional harness integration. New reusable and cache-warming workflows build, test, verify, cache, and report integration artifacts. ChangesIntegration-aware CI and cache workflows
Estimated code review effort: 4 (Complex) | ~60 minutes Sequence Diagram(s)sequenceDiagram
participant ChangedWorkerDiscovery
participant CIWorkflow
participant HarnessIntegrationWorkflow
participant IntegrationEngine
ChangedWorkerDiscovery->>CIWorkflow: emit integration_changed
CIWorkflow->>HarnessIntegrationWorkflow: invoke when integration_changed is true
HarnessIntegrationWorkflow->>IntegrationEngine: checkout and build pinned revision
HarnessIntegrationWorkflow->>HarnessIntegrationWorkflow: run harness and Console E2E tests
HarnessIntegrationWorkflow->>CIWorkflow: publish verified reports and artifacts
Poem
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 Checkov (3.3.8).github/workflows/_harness-integration.ymlTraceback (most recent call last): .github/workflows/cache-warm.ymlTraceback (most recent call last): .github/workflows/ci.ymlTraceback (most recent call last): 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 |
skill-check — worker0 verified, 47 skipped (no docs/).
Four for four. Nicely done. |
Summary
Why
Pull-request caches were scoped to individual merge refs and could not be reused by sibling PRs or main. The repository had more than 11 GB of active caches against a 10 GiB operating budget, while a cache miss pushed the integration job beyond 20 minutes.
The default branch did not run the main CI workflow after merges, so it never produced reusable caches. Harness changes also expanded every runtime dependency from iii.worker.yaml into the worker matrices, causing unrelated jobs such as shell formatting to run even though those workers do not compile against the local harness source.
This change makes main the cache producer, keeps PR jobs restore-only, and separates integration-stack coverage from per-worker checks.
Validation
Fixes MOT-4183
Refs MOT-4107
Summary by CodeRabbit
New Features
Bug Fixes