Docker image unpacker#715
Merged
Merged
Conversation
Member
|
Is there a technical reason this depends on those other branches, or is it just a result of your local git tree? I'm assuming the latter, but want to know what I should look out for when reviewing. |
Collaborator
Author
|
No, it doesn't technically depend on those other branches, it's just that I need all these features for the project I'm currently working on. |
added 4 commits
March 25, 2026 18:25
added 7 commits
May 4, 2026 14:59
rbs-jacob
requested changes
May 14, 2026
rbs-jacob
left a comment
rbs-jacob
left a comment
Member
There was a problem hiding this comment.
Some small stuff, then this should be good to go!
rbs-jacob
approved these changes
May 18, 2026
rbs-jacob
left a comment
rbs-jacob
left a comment
Member
There was a problem hiding this comment.
Pending passing tests, this looks good enough to me for now. May need to make some adjustments as we encounter more Docker images in the wild, but that shouldn't block merging this at the moment.
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
3 participants
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.
One sentence summary of this PR (This should go in the CHANGELOG!)
Docker image tarball unpacker.
Link to Related Issue(s)
Please describe the changes in your request.
Adds a Docker image unpacker: unpack a docker image tar ball to a filesystem that represents the filesystem from this image once run as a container. This unpacker walks the different image layers and applies them, including whiteouts and opaque whiteouts.
Anyone you think should look at this, specifically?
No