Skip to content

exporter/containerimage: support unpack for rewrite-timestamp mode#6939

Draft
AkihiroSuda wants to merge 1 commit into
moby:masterfrom
AkihiroSuda:fix-4230
Draft

exporter/containerimage: support unpack for rewrite-timestamp mode#6939
AkihiroSuda wants to merge 1 commit into
moby:masterfrom
AkihiroSuda:fix-4230

Conversation

@AkihiroSuda

Copy link
Copy Markdown
Member

ImageWriter.Commit now returns the committed remotes (the rewritten layers when rewrite-timestamp is set) as Result[Remote], and unpackImage consumes them instead of re-fetching the original (non-rewritten) layers from the source ref. The provider of a rewritten remote preserves the Unlazier of the original provider, so that non-rewritten lazy layers can still be fetched before unpacking.

Fixes #4230

Note: used Claude

ImageWriter.Commit now returns the committed remotes (the rewritten
layers when rewrite-timestamp is set) as Result[Remote], and unpackImage
consumes them instead of re-fetching the original (non-rewritten) layers
from the source ref. The provider of a rewritten remote preserves the
Unlazier of the original provider, so that non-rewritten lazy layers can
still be fetched before unpacking.

Fixes issue 4230

Co-Authored-By: Claude Fable 5 <[email protected]>
Signed-off-by: Akihiro Suda <[email protected]>
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.

exporter/containerimage: support unpack for rewrite-timestamp mode

1 participant