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

Revert "Revert "Attempt to speed up deploy"" #60

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

tianon
Copy link
Member

@tianon tianon commented Jun 26, 2024

This reverts commit 4473698.

A little bit byzantine, but this is #56 re-applied (which was reverted in #58). See #58 (comment) (and preceding comments) for debugging/justification (and why this should be safe to re-apply).

@tianon tianon requested a review from yosifkit as a code owner June 26, 2024 23:14
@tianon tianon merged commit 72d39b3 into docker-library:main Jun 26, 2024
1 check passed
@tianon tianon deleted the always-head-again branch June 26, 2024 23:29
@tianon
Copy link
Member Author

tianon commented Jun 26, 2024

The final tally is that our deploy-riscv64 build is ~20s faster with this change (~3m20s before, ~3m after).

Not a windfall, but definitely measurably better. ❤️

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