Skip to content

dockerfile: add --unpack flag to ADD #5991

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

Merged
merged 1 commit into from
May 26, 2025
Merged

Conversation

tonistiigi
Copy link
Member

Currently, the default behavior is to extract archives from local paths and keep original archive on URL paths.

This keeps the default but allows setting explicit --unpack=bool flag for the opposite behavior.

Fixes #4482

Currently, the default behavior is to extract archives from
local paths and keep original archive on URL paths.

This keeps the default but allows setting explicit --unpack=bool
flag for the opposite behavior.

Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
@crazy-max crazy-max added this to the v0.23.0 milestone May 26, 2025
@crazy-max crazy-max merged commit 71da7ab into moby:master May 26, 2025
139 checks passed
@fenollp
Copy link
Contributor

fenollp commented May 26, 2025

Hello! Looks like the post-merge CI failed and a new labs image was not pushed: https://github.com/moby/buildkit/actions/runs/15249959528/job/42885256145

Here shows a 429: https://github.com/moby/buildkit/actions/runs/15249959520/job/42884278789#step:3:1654

Please push a labs tag 🙏

@crazy-max
Copy link
Member

Hello! Looks like the post-merge CI failed and a new labs image was not pushed: https://github.com/moby/buildkit/actions/runs/15249959528/job/42885256145

Here shows a 429: https://github.com/moby/buildkit/actions/runs/15249959520/job/42884278789#step:3:1654

Please push a labs tag 🙏

Ah yes this is related to GitHub infra issue: https://www.githubstatus.com/incidents/d0nm3xcdc5jw
Will re-run the workflows

@tonistiigi
Copy link
Member Author

tonistiigi commented May 27, 2025

Please push a labs tag

FYI, this is not labs-only. @AkihiroSuda should it? or is it obvious enough?

@AkihiroSuda
Copy link
Member

Seems safe to graduate from labs at birth

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dockerfile: "--unpack" option for ADD
4 participants