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

chore(ci): Speed up critical path of CI #5275

Merged
merged 1 commit into from Jan 8, 2024
Merged

chore(ci): Speed up critical path of CI #5275

merged 1 commit into from Jan 8, 2024

Conversation

epage
Copy link
Member

@epage epage commented Dec 28, 2023

We've decided to try using some of our funding to speed up CI. kbknapp has experience with buildjet in the past which allows us to keep our Actions and switch out our runners.

As we are charged for num_cores * time, increasing core counts could decrease time, both helping us and keeping costs down. I chose 8 cores (an upgrade over ubuntu-latests 2 cores) as kbknapp knew someone who benchmarked things for Rust/Python and found that a good fit.

I only switched a subset of jobs over to buildjet to focus on jobs where
most of the time is spent on highly parallelizable operations.
Buildjet dropped our Linux test jobs from 8-9min to 2-3min.
The checks and UI-test jobs only improved by 30s-1min each, so I left
them out.
We can iterate as we go.

We've decided to try using some of our funding to speed up CI.
kbknapp has experience with buildjet in the past which allows us to keep
our Actions and switch out our runners.

As we are charged for `num_cores * time`, increasing core counts could
decrease time, both helping us and keeping costs down.
I chose 8 cores (an upgrade over `ubuntu-latest`s 2 cores) as kbknapp
knew someone who benchmarked things for Rust/Python and found that a
good fit.

I only switched a subset of jobs over to buildjet to focus on jobs where
most of the time is spent on highly parallelizable operations.
Buildjet dropped our Linux test jobs from 8-9min to 2-3min.
The checks and UI-test jobs only improved by 30s-1min each, so I left
them out.
We can iterate as we go.
@epage epage merged commit c603f34 into clap-rs:master Jan 8, 2024
21 checks passed
@epage epage deleted the ci branch January 8, 2024 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant