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

wip: fast build only amd64 #156

Closed
wants to merge 1 commit into from
Closed

wip: fast build only amd64 #156

wants to merge 1 commit into from

Conversation

corneliusroemer
Copy link
Member

@corneliusroemer corneliusroemer commented May 16, 2023

Description of proposed changes

QEMU arm builds slow builds down a lot. With AMD only, CI would take around 10 min, with ARM as well, it's 60min.

For faster feedback loops it could make sense to be able to run CI with AMD only.

The way I've set it up is definitely not good, but it's a POC. Could be helpful for @joverlee521 while working on #155 for example.

@tsibley
Copy link
Member

tsibley commented May 22, 2023

Closing since we're not going to use this PR as-is.

We definitely would benefit from a faster CI feedback loop, but I think routinely disabling one half of the build would introduce other bugs. Would be good to do more thinking about how to make platforms an optional switch for some PRs, those where it would specifically be beneficial. One alternative way is temporary commits on the PR branch that edit the platforms used, which are then dropped and retested with full CI before merge.

@tsibley tsibley closed this May 22, 2023
@victorlin victorlin deleted the fast-build branch June 18, 2024 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

2 participants