These jobs can likely use reduced runner sizes to avoid wasting our CPU quota, as much of the long-running part of the job is single-threaded.
This will also give us more (job) parallelisem from the same number of CPU that we are using.
Suggested in: #32989 (review)
These jobs can use reduced runner size to avoid wasting CPU, as much of
the long-running part of the job is single-threaded.
Suggested in: https://github.com/bitcoin/bitcoin/pull/32989#discussion_r2321775620
Co-authored-by: MarcoFalke <*~=`'#}+{/-|&$^_@721217.xyz>
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/33319.
<!--021abf342d371248e50ceaed478a90ca-->
See the guideline for information on the review process.
| Type | Reviewers |
|---|---|
| ACK | janb84, kevkevinpal, m3dwards |
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
<!--174a7506f384e20aa4161008e828411d-->
Reviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
ACK 5eeb2facbbbbf68a2c30ef9e6747e39c85d7b116
PR reduces select container sizes / cirrus runners. Not using unnecessary "large" runners is a good idea.
ACK 5eeb2fa
Looks good to me, makes sense to me to use smaller runners. All the checks passed on this PR and the longest running one only too 44 minutes
is there any reason to keep CentOS, depends, gui and MSan, depends on a lg runner?
ACK 5eeb2facbbbbf68a2c30ef9e6747e39c85d7b116
WIP stats collection (forked from @0xB10C project) here: https://willcl-ark.github.io/bitcoin-core-ci-stats