Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: 💚 Reduce self-hosted runner usage #869

Draft
wants to merge 35 commits into
base: master
Choose a base branch
from

Conversation

timbrinded
Copy link
Contributor

@timbrinded timbrinded commented Feb 11, 2025

Changes

  • Moved clippy as dependent on build, so that it can use the cached results from build step
  • Split rust-test into generating the test files and then execution step. Our runners have too slow network speed so do not benefit from sharding
  • Made zombienet upgrade task a GHA runner step, since it only spawns 3 nodes

Context

The queue to use self-hosted runners can sometimes be over an hour long.

This has introduced a blockages into the critical path, which can be ameliorated by lowering the usage of self-hosted for heavy tasks only.

@timbrinded timbrinded added B0-silent Changes should not be mentioned in any release notes D2-notlive PR doesn't change runtime/client code labels Feb 11, 2025
Copy link
Contributor

github-actions bot commented Feb 11, 2025

Coverage Report

(master)

@@                    Coverage Diff                    @@
##           master   ci/distribute-runners      +/-   ##
=========================================================
+ Coverage   65.51%                  65.97%   +0.46%     
- Files         343                     339       -4     
- Lines       59504                   59084     -420     
=========================================================
- Hits        38982                   38978       -4     
- Misses      20522                   20106     -416     
Files Changed Coverage
/chains/container-chains/runtime-templates/frontier/src/lib.rs 57.65% (-0.05%)
/chains/container-chains/runtime-templates/simple/src/lib.rs 87.25% (-0.02%)
/chains/orchestrator-paras/runtime/dancebox/src/lib.rs 87.98% (-0.01%)
/chains/orchestrator-paras/runtime/flashbox/src/lib.rs 46.80% (-0.05%)

Coverage generated Fri Feb 14 21:11:47 UTC 2025

Copy link
Contributor

github-actions bot commented Feb 11, 2025

WASM runtime size check:

Compared to target branch

dancebox runtime: 1416 KB (no changes) ✅

flashbox runtime: 824 KB (no changes) ✅

dancelight runtime: 2172 KB (+2172 KB) ⚠️

container chain template simple runtime: 1116 KB (-4304 KB) ✅

container chain template frontier runtime: 1404 KB (-5156 KB) ✅

@tmpolaczyk
Copy link
Contributor

@timbrinded if nextest is giving problems, can you split that into a future PR? The other changes are already very valuable, I don't want to be blocked by that

@timbrinded
Copy link
Contributor Author

timbrinded commented Feb 14, 2025

I'm very close, let me cook @tmpolaczyk

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B0-silent Changes should not be mentioned in any release notes D2-notlive PR doesn't change runtime/client code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants