ci: Use Cirrus "greedy" flag to use idle CPU time when available #23797

pull luke-jr wants to merge 1 commits into bitcoin:master from luke-jr:cirrus_greedy changing 1 files +4 −4

luke-jr commented at 7:34 PM on December 16, 2021: member

Disabled for MSan to avoid excessive RAM usage
laanwj added the label Tests on Dec 16, 2021
luke-jr commented at 3:58 AM on December 17, 2021: member

CI failure seems unrelated?

Aside from that, the CI durations seem to be much improved.
in .cirrus.yml:223 in a48947d20a outdated
```
 219 |  task:
 220 |    name: '[MSan, depends] [focal]'
 221 |    << : *GLOBAL_TASK_TEMPLATE
 222 |    container:
 223 |      image: ubuntu:focal
 224 | +    greedy: false
```
MarcoFalke commented at 8:52 AM on December 17, 2021:

I think this can be removed. I was wrong yesterday to say that greedy can lead to OOM. It can not, since we always set the jobs with MAKEJOBS. So only MAKEJOBS can lead to OOM.

luke-jr commented at 6:18 PM on December 17, 2021:

Done
MarcoFalke commented at 9:04 AM on December 17, 2021: member

https://medium.com/cirruslabs/introducing-greedy-container-instances-29aad06dc2b4
luke-jr force-pushed on Dec 17, 2021
MarcoFalke commented at 9:07 AM on December 18, 2021: member

asan OOM
ci: Use Cirrus "greedy" flag to use idle CPU time when available
MAKEJOBS limited for MSan to avoid excessive RAM usage
cfeb1942bc
luke-jr force-pushed on Dec 18, 2021
luke-jr commented at 5:32 PM on December 18, 2021: member

The rest of the tasks look ok, so limited that one to -j4 also
MarcoFalke commented at 11:01 AM on December 20, 2021: member

LGTM. This is hard to test, but easy to revert, so might be good to go.
MarcoFalke merged this on Dec 20, 2021
MarcoFalke closed this on Dec 20, 2021
sidhujag referenced this in commit f1302beb4d on Dec 20, 2021
DrahtBot locked this on Dec 20, 2022

Contributors

luke-jr

MarcoFalke

Labels

Tests