vendor: github.com/moby/go-archive v0.3.1 - #53260
Merged
Merged
Conversation
full diff: moby/go-archive@v0.3.0...v0.3.1 Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
There was a problem hiding this comment.
Pull request overview
This PR updates the vendored github.com/moby/go-archive dependency to v0.3.1 to address #53257, a regression where extracting layers can fail when a tar contains a directory entry whose parent directory is implied (not explicitly present in the archive).
Changes:
- Vendor bump
github.com/moby/go-archivefrom v0.3.0 to v0.3.1 acrossgo.mod,go.sum, andvendor/modules.txt. - Fix implied-directory handling during extraction by creating parent directories for all entry types (including directory entries), preventing
mkdir ...: no such file or directoryfailures. - Reduce noisy error logging during tarball creation by ignoring
io.ErrClosedPipefrom close operations when the read-end is closed early.
Reviewed changes
Copilot reviewed 1 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| vendor/github.com/moby/go-archive/archive.go | Applies upstream fix for implied parent directories during extraction; adjusts Close error logging behavior. |
| vendor/modules.txt | Updates vendored module version entry for github.com/moby/go-archive to v0.3.1. |
| go.mod | Bumps required github.com/moby/go-archive dependency to v0.3.1. |
| go.sum | Updates checksums for github.com/moby/go-archive v0.3.1. |
chhe
pushed a commit
to chhe/act_runner
that referenced
this pull request
Aug 8, 2026
Revert https://gitea.com/gitea/runner/pulls/1130. Docker 29.7.1 fixed both regressions it worked around, moby/moby#53261 and moby/moby#53260, so only 29.7.0 still needs it. Verified live with a relative and an absolute `/var/run` symlink: without the workaround the copy passes on 29.4.0, 29.6.2 and 29.7.1, and fails on 29.7.0 alone. Fixes: https://gitea.com/gitea/runner/issues/1131 Reviewed-on: https://gitea.com/gitea/runner/pulls/1155 Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com> Co-authored-by: silverwind <me@silverwind.io>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
full diff: moby/go-archive@v0.3.0...v0.3.1
Summary
Release notes (optional)
A picture of a cute animal (not mandatory but encouraged)