Skip to content

dockerfile: detect zstd archive contexts - #7069

Merged
crazy-max merged 1 commit into
moby:masterfrom
crazy-max:add-zstd-archive-context-cap
Aug 26, 2026
Merged

dockerfile: detect zstd archive contexts#7069
crazy-max merged 1 commit into
moby:masterfrom
crazy-max:add-zstd-archive-context-cap

Conversation

@crazy-max

Copy link
Copy Markdown
Member

picked from #7034 (comment)

This adds zstd to Dockerfile archive context detection so zstd-compressed tar archives are recognized the same way as gzip, bzip2, xz, and uncompressed tar archives. The Dockerfile frontend now advertises this behavior with a dedicated frontend capability.

@crazy-max crazy-max added this to the v0.33.0 milestone Aug 24, 2026
@crazy-max
crazy-max force-pushed the add-zstd-archive-context-cap branch from b682321 to a8b22b6 Compare August 24, 2026 12:13
@crazy-max
crazy-max requested a review from tonistiigi August 24, 2026 12:14
@crazy-max
crazy-max marked this pull request as ready for review August 24, 2026 12:14
@crazy-max
crazy-max force-pushed the add-zstd-archive-context-cap branch 2 times, most recently from e274ed5 to 2b6910c Compare August 24, 2026 13:12

@tonistiigi tonistiigi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@crazy-max
crazy-max force-pushed the add-zstd-archive-context-cap branch from 2b6910c to 4081fec Compare August 26, 2026 08:53
@crazy-max

Copy link
Copy Markdown
Member Author

Approved but check this https://github.com/moby/go-archive/blame/main/compression/compression_detect.go#L49-L65

good catch, added zstd skippable frames detection as well.

@crazy-max
crazy-max merged commit ecf067f into moby:master Aug 26, 2026
233 of 234 checks passed
@crazy-max
crazy-max deleted the add-zstd-archive-context-cap branch August 26, 2026 09:45
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.

2 participants