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

only build for linux/amd64 on PRs #2460

Merged
merged 2 commits into from May 30, 2023

Conversation

imjasonh
Copy link
Collaborator

@imjasonh imjasonh commented Apr 3, 2023

Building for all platforms takes more than an hour on a single-core runner. We don't actually push the resulting image anywhere anyway, it's just a quick check that the image build will work. If there's a platform-specific issue in a future PR we can disable this check while we diagnose it.

Before this PR, building the executor image for all platforms takes 1h0m38s: https://github.com/GoogleContainerTools/kaniko/actions/runs/4597484542/jobs/8120251637?pr=2459

With this PR, building the executor image takes 1m47s: https://github.com/GoogleContainerTools/kaniko/actions/runs/4598191271/jobs/8121901340?pr=2460

@aaron-prindle
Copy link
Collaborator

+1 thanks for this 🙏 ! If you can rebase this, I'm happy to help get this in

Copy link
Collaborator

@aaron-prindle aaron-prindle left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@aaron-prindle aaron-prindle merged commit 971b9f4 into GoogleContainerTools:main May 30, 2023
10 checks passed
kylecarbs pushed a commit to coder/kaniko that referenced this pull request Jul 12, 2023
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.

None yet

2 participants