ci: use larger runners - Githubissues

penumbra-zone / decaf377

decaf377 is a prime-order group designed for use in SNARKs over BLS12-377

https://protocol.penumbra.zone/main/crypto/decaf377.html

12 stars 12 forks source link

ci: use larger runners #83

Closed conorsch closed 5 months ago

conorsch commented 5 months ago

The longest running CI jobs are the test suite, predictably. Before this change, runtimes were:

26m for r1cs
1h26m for r1cs,u32_backend

The other, non-test-suite jobs are all sub-minute, likely due to caching. To get runtimes down, we:

use larger buildjet runners with more vcpus
switch from "cargo test" to "cargo nextest"
update caching logic to use the astria helper (significantly more headroom on caches)

conorsch commented 5 months ago

CI runtime down 76m -> 10m on fairly naive changes. I migrated to cargo nextest but otherwise tried to preserve the existing test functionality, including the features matrix. One thing worth considering is running the test suite in --release mode. We've not been doing that on this repo, but presumably we should be. Anyway, no need to block on that, this seems worthwhile to merge without further rabbitholing.