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

ci(runners): use larger github hosted runners + reduce cron frequency #9002

Merged
merged 3 commits into from
Sep 29, 2023

Conversation

joshua-goldstein
Copy link
Contributor

@joshua-goldstein joshua-goldstein commented Sep 24, 2023

Description: Currently we use large 32gb self-hosted runners for jobs that require more resources. Github now offers their own suite of hosted runners. We can now transition to these new runners. We also reduce the frequency of some scheduled jobs and remove some particular workflows from each PR run.

@ghost ghost added the area/integrations Related to integrations with other projects. label Sep 24, 2023
Copy link
Contributor

@all-seeing-code all-seeing-code left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we retire the auto scaler as we merge these PRs

@joshua-goldstein
Copy link
Contributor Author

Should we retire the auto scaler as we merge these PRs

The coordinator ec2 instance is pretty small (I believe micro size) so it doesn't cost much to keep that infrastructure in place. But yeah, it will certainly pick up less jobs and cost less on the ec2 side of things.

However there are no github hosted arm64 ubuntu runners available, so if we want to keep running the arm workflow we would have to keep that in place.

@joshua-goldstein joshua-goldstein changed the title ci(runners): use larger github hosted runners ci(runners): use larger github hosted runners + reduce cron frequency Sep 26, 2023
@joshua-goldstein joshua-goldstein merged commit 6dc93c7 into main Sep 29, 2023
@joshua-goldstein joshua-goldstein deleted the joshua/ci-runners branch September 29, 2023 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/integrations Related to integrations with other projects.
Development

Successfully merging this pull request may close these issues.

2 participants