Skip to content

Conversation

@mtrmac
Copy link
Collaborator

@mtrmac mtrmac commented Mar 26, 2025

This is a backport of #2795.

Marked as draft, pending a decision on whether we can include this. Cc: @TomSweeneyRedHat

mtrmac added 3 commits March 26, 2025 20:22
Move the layerType value closer to the code that
actually knows the format.

Don't allocate/initialize a sha256 state if we don't
know we are going to need it.

Should not change behavior.

Signed-off-by: Miloslav Trmač <[email protected]>
Should not change behavior.

Signed-off-by: Miloslav Trmač <[email protected]>
Correctly compute the DiffID of non-gzip-compressed layers,
and if possible, set the layer MIME type correctly.

Signed-off-by: Miloslav Trmač <[email protected]>
@mtrmac mtrmac changed the base branch from main to release-5.34 March 26, 2025 19:25
@TomSweeneyRedHat
Copy link
Member

LGTM
Before merging, let's go some RHEL Jira cards setup

@TomSweeneyRedHat TomSweeneyRedHat marked this pull request as ready for review March 27, 2025 13:21
Copy link
Member

@Luap99 Luap99 left a comment

Choose a reason for hiding this comment

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

LGTM

@Luap99 Luap99 merged commit 6b6fd18 into containers:release-5.34 Mar 27, 2025
10 checks passed
@mtrmac mtrmac deleted the tarball-compression-5.34 branch March 27, 2025 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants