Skip to content

ci: use larger Depot Linux runners#1023

Open
indygreg wants to merge 1 commit intomainfrom
gps-depot-larger-runners
Open

ci: use larger Depot Linux runners#1023
indygreg wants to merge 1 commit intomainfrom
gps-depot-larger-runners

Conversation

@indygreg
Copy link
Collaborator

Default runners have 2 CPUs. We can achieve a nice speed-up (but not quite linear) by leveraging runners with more CPUs.

Default runners have 2 CPUs. We can achieve a nice speed-up
(but not quite linear) by leveraging runners with more CPUs.
@indygreg
Copy link
Collaborator Author

With the 4 CPU runners (sample size of 1):

  • Before: 40m wall; 8971m machine time
  • After: 29m wall; 5318m machine time

Perfect scaling of machine time would be 4486m. Since cost scales linearly with CPU count, this change will increase costs by ~18%.

#1022 will help with CPU scaling by making PGO+BOLT run tests in parallel. So probably looking at a 10-15% cost premium after that merges.

IMO this change is worth it since people time costs more than machine time and faster CI enables people to be more productive.

I haven't tested 8 CPU runners thoroughly. But they do yield additional speed-ups.

@indygreg indygreg marked this pull request as ready for review March 21, 2026 08:37
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.

1 participant