Repository navigation
perf(ci): overlap independent steps and reduce cache overhead - #6072
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Merge Risk: ⚪ Minimal · up to
|
| Check name | Status | Explanation |
|---|---|---|
| Description Check |  |
Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | |
The title clearly and concisely summarizes the main changes: parallelizing independent CI steps and reducing cache overhead. |
| Docstring Coverage | |
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
✨ Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- 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.
Comment @coderabbitai help to get the list of available commands.
Deploying egg-v3 with
|
| Latest commit: |
27e35a7
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://2dddb3c6.egg-v3.pages.dev |
| Branch Preview URL: | https://perf-parallel-ci-setup.egg-v3.pages.dev |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6072 +/- ##
==========================================
+ Coverage 83.68% 84.02% +0.34%
==========================================
Files 734 734
Lines 22639 22639
Branches 4560 4560
==========================================
+ Hits 18945 19023 +78
+ Misses 3193 3125 -68
+ Partials 501 491 -10 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Deploying egg with
|
| Latest commit: |
27e35a7
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://4d9484c2.egg-cci.pages.dev |
| Branch Preview URL: | https://perf-parallel-ci-setup.egg-cci.pages.dev |
fengmk2
changed the title
|
@codex review |
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.
|
Codex Review: Didn't find any major issues. You're on a roll. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
CI waits for independent work and repeats cache work across jobs. These waits delay tests.
Run service startup, tool setup, and cache restoration in parallel. Compute the dependency hash once before installation. Use one writer per shared cache key, and skip saves after exact cache hits.
Start ecosystem checkout and coverage downloads early. Wait where their results are first required. Upload main-suite results while example tests run. Build packages and the site in parallel after static checks.
Tests still wait for service startup. Windows
Chocolateyinstallations remain sequential because they share installation state.This change applies the approaches from Oxc #27476 and Oxc #27482.
The baseline CI run, setup/cache CI run, and current CI run use the same PR matrix.
Runner time sums job durations without billing weights. Setup starts when checkout finishes and ends when tests start. Workflow elapsed time includes scheduling waits.
Against the baseline, setup time fell 23.9%, and CI elapsed time fell 4.8%. The current CI run took 50 seconds longer than the setup/cache run. Main-test execution increased by 4m36s across nine jobs between those runs.
The package/site build span fell from 30s to 17s after the builds ran in parallel. The package build still took 13s. The site build still took 17s.
The
cnpmcoreE2E test step fell from 5m29s to 4m14s between the last two runs. Its setup fell from 27s to 24s. The faster tests account for most of the E2E reduction.These measurements use one run per version, with different commits and runner allocation. They do not establish a controlled benchmark or a guaranteed speed increase.
Summary by CodeRabbit