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

optimize build time by using cross-compilation instead of emulation #52

Merged
merged 2 commits into from
Apr 10, 2024

Conversation

thomasferrandiz
Copy link
Contributor

This PR should reduce the build time on arm64 from 14 mn to 2mn.

It is based-off #50

Copy link
Contributor

@manuelbuil manuelbuil left a comment

Choose a reason for hiding this comment

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

All the additions in the Dockerfile, shouldn't they be only for arm64?

Dockerfile Show resolved Hide resolved
Dockerfile Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
@thomasferrandiz
Copy link
Contributor Author

All the additions in the Dockerfile, shouldn't they be only for arm64?

No the point is to have the same Dockerfile optimised for all arch, that's why we use BUILDPLATFORM and TARGETPLATFORM

@thomasferrandiz thomasferrandiz force-pushed the optimize-build branch 2 times, most recently from 36bbc03 to 18a2648 Compare April 9, 2024 07:44
Copy link
Contributor

@manuelbuil manuelbuil left a comment

Choose a reason for hiding this comment

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

Some changes needed to consume secrets in the new EIO way

.github/workflows/image-push.yml Show resolved Hide resolved
.github/workflows/image-push.yml Outdated Show resolved Hide resolved
@thomasferrandiz
Copy link
Contributor Author

Some changes needed to consume secrets in the new EIO way

It should be fixed now

@thomasferrandiz thomasferrandiz merged commit 8cf0620 into rancher:master Apr 10, 2024
3 checks passed
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.

2 participants