perf: speed up CI tests with shared Docker image and more shards#161
Merged
perf: speed up CI tests with shared Docker image and more shards#161
Conversation
- Build Docker image once in a dedicated job, share via artifact (saves ~4min of redundant Docker builds per shard) - Increase test shards from 3 to 5 for better distribution - Add separate test-unit job that runs without Docker overhead - Reduce test/hook timeouts from 120s to 60s
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
Comment on lines
+214
to
+218
| - name: Download Docker image | ||
| uses: actions/download-artifact@v4 | ||
| with: | ||
| name: docker-image | ||
|
|
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Neovim v0.11.6 ships official prebuilt tarballs for Linux x86_64. Replaces the entire multi-stage build (clone + cmake + compile) with a single curl | tar (~10MB download, seconds instead of minutes).
- Default client timeout 30s → 120s (workspace creation takes 30-60s with Docker operations, credential sync, and agent setup) - Skip agent binary updates (curl install scripts) in test environments via SKIP_AGENT_UPDATES=true — agents are already in the Docker image - Fixes flaky "Request timed out" failures in CI integration tests
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
CI tests are slow due to redundant Docker image builds and an unnecessarily heavy Neovim compile-from-source step. This PR fixes both.
Before (from recent successful run):
Changes:
dockerjob, save as artifact,docker loadin each consumer job. Saves ~4min per shard.test-unitjob — Unit tests run in parallel without Docker overhead at all.Test plan
nvim --versionshows 0.11.6)🤖 Generated with Claude Code