ci: reduce GitHub Actions costs #2177
Merged
+107
−121
Conversation
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
- Add concurrency groups to cancel in-progress runs on new commits - Switch Test, Clippy, six-peer tests from freenet-default-runner to ubuntu-latest - Remove redundant cancel-workflow-action steps (native concurrency handles this) - Reduce six_peer_regression timeout from 120m to 30m - Skip cross-compile on PRs (only run on main/tags) Estimated savings: ~50% reduction in CI costs by using standard runners instead of larger runners, and avoiding duplicate work. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Nova has 128GB RAM and fast network, configured with mold linker and sccache for fast builds. Eliminates GitHub Actions billing entirely. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Jobs now run in order from fastest to slowest: 1. Conventional Commits (~5s) 2. Fmt (~10s) 3. Clippy (~3-5min) 4. Test (~10-15min) 5. six-peer tests (after Test) This ensures simple issues like formatting or commit messages fail immediately rather than waiting for long builds. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Add `needs` dependencies to chain jobs in order of execution time: 1. Conventional Commits (fastest, ~5s) 2. Fmt (needs Conventional Commits, ~10s) 3. Clippy (needs Fmt, ~3-5min) 4. Test (needs Clippy, ~10-15min) 5. six-peer tests (needs Test) With a single self-hosted runner, this ensures faster checks run first and fail fast, avoiding wasted time on slow tests when simple checks would fail anyway. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Remove /tmp/freenet and /tmp/freenet-* directories before running tests to prevent permission denied errors when the self-hosted runner user cannot access directories created by other users. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Problem
Hit the $300/month GitHub Actions budget. Analysis showed expensive jobs running on
freenet-default-runner(larger GitHub-hosted runner).Solution
Set up a self-hosted runner on nova (128GB RAM, fast network) and switched CI to use it.
Changes
Self-hosted runner - Configured
nova-runnerwith labelsself-hosted, Linux, X64, novaNative concurrency cancellation - Added
concurrencygroups to auto-cancel in-progress runs when new commits pushReduced timeout -
six_peer_regressiontimeout reduced from 120m → 30mSkip cross-compile on PRs - Release binaries only need to build on main/tags
Cost Impact
Testing
This PR will test itself on the new nova runner.
[AI-assisted - Claude]