ockam icon indicating copy to clipboard operation
ockam copied to clipboard

Try to speed up CI by splitting testing of crates

Open mrinalwadhwa opened this issue 2 years ago • 3 comments

Rust - Test & Rust - Nightly Test jobs take significantly more time than the other jobs in the Rust workflow.

Screen Shot 2022-06-02 at 11 55 28 PM

It would be interesting to try if we had one job per crate that did this.

cd implementation/rust/ockam/{crate_name} && cargo test

And the rust workflow ran them in parallel.

Would it speed up time spent in CI?

mrinalwadhwa avatar Jun 03 '22 07:06 mrinalwadhwa

Would you mind assigning this task to me?

Shreyan11 avatar Jun 29 '22 10:06 Shreyan11

@Shreyan11 that would be amazing. Thank you for picking it up 🙏

mrinalwadhwa avatar Jun 29 '22 14:06 mrinalwadhwa

nextest prints a nice report https://github.com/build-trust/ockam/issues/2924 and I noticed that a big chunk of time spent in the Rust Test job is spent testing our examples, so splitting that into a separate job/s should reduce time significantly https://github.com/build-trust/ockam/tree/develop/examples/rust

mrinalwadhwa avatar Jul 05 '22 19:07 mrinalwadhwa

@metaclips should we close this?

adrianbenavides avatar Apr 10 '23 11:04 adrianbenavides

No, we still limit concurrency when running the examples, we will need to revisit the example code.

metaclips avatar Sep 11 '23 11:09 metaclips

I am closing this issue in favour of https://github.com/build-trust/ockam/issues/3619 and https://github.com/build-trust/ockam/issues/3005

metaclips avatar Sep 11 '23 12:09 metaclips