Skip to content

ci: use larger Depot Linux runners#1023

Merged
zanieb merged 1 commit intomainfrom
gps-depot-larger-runners
Mar 23, 2026
Merged

ci: use larger Depot Linux runners#1023
zanieb merged 1 commit intomainfrom
gps-depot-larger-runners

Conversation

@indygreg
Copy link
Copy Markdown
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
Copy Markdown
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
@geofft
Copy link
Copy Markdown
Collaborator

geofft commented Mar 23, 2026

I defer to @zanieb who knows more about our Depot setup and costs, but I would say yes, the turnaround time for getting a Python release out is very important and currently that requires two CI runs (on PR and on HEAD) so paying to get that cut down seems valuable.

(This is probably overengineering but I would likely be in favor of using even larger runners conditionally if the commit bumps a CPython patch version, for the same reason.)

@zanieb zanieb merged commit 7f200fd into main Mar 23, 2026
548 of 549 checks passed
@zanieb zanieb deleted the gps-depot-larger-runners branch March 23, 2026 14:49
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.

3 participants