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

(gha) use 32core runner for "rake checks" #1060

Draft
wants to merge 1 commit into
base: production
Choose a base branch
from

Conversation

jhoblitt
Copy link
Member

@jhoblitt jhoblitt commented Feb 20, 2024

Reduces run time from ~8 mins -> ~5mins.

@jhoblitt jhoblitt added the enhancement New feature or request label Feb 20, 2024
@jhoblitt jhoblitt force-pushed the maint/rake_checks-speed branch from fbd1c46 to acfb5fc Compare February 20, 2024 23:06
@jhoblitt jhoblitt changed the title (TESTING) 32cores (gha) use 32core runner for "rake checks" Feb 20, 2024
@jhoblitt jhoblitt force-pushed the maint/rake_checks-speed branch from acfb5fc to 5193e4c Compare February 20, 2024 23:07
@jhoblitt jhoblitt marked this pull request as draft February 20, 2024 23:08
@jhoblitt
Copy link
Member Author

I'm unsure of how much of win this ends up being as it appears to take > 1min for a 32c runner to come online.

@jhoblitt jhoblitt force-pushed the maint/rake_checks-speed branch from 5193e4c to 19bbe2c Compare February 20, 2024 23:10
Reduces run time from ~8 mins -> ~5mins.
@jhoblitt jhoblitt force-pushed the maint/rake_checks-speed branch from 19bbe2c to 0a39fae Compare August 5, 2024 22:38
@jhoblitt
Copy link
Member Author

jhoblitt commented Aug 5, 2024

This reduces the current time from ~6mins to ~4mins. However, the total wallclock time seems closer to 6mins as it seems to take longer to get a 32c runner.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant