Skip to content

exporter: skip empty attestation manifests - #7419

Open
crazy-max wants to merge 1 commit into
moby:masterfrom
crazy-max:fix-attestation-empty-layers
Open

crazy-max wants to merge 1 commit into
moby:masterfrom
crazy-max:fix-attestation-empty-layers

Conversation

@crazy-max

@crazy-max crazy-max commented Nov 12, 2026 •

Copy link
Copy Markdown
Member

Skip attestation export when unbundling produces no entries. An empty bundle has nothing to attest, so avoid writing its manifest and config.

@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.

Why would we make attestation manifest at all if it doesn't container any items?

Skip attestation export when unbundling produces no entries. An empty
bundle has nothing to attest, so avoid writing its manifest and config.

Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@crazy-max
crazy-max force-pushed the fix-attestation-empty-layers branch from 2a062f7 to c6fca9c Compare November 13, 2026 08:28
@crazy-max crazy-max changed the title exporter: fix empty attestation manifest layers exporter: skip empty attestation manifests Nov 13, 2026
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.

Attestation manifest can still serialize "layers": null when a platform has an empty attestation list (follow-up to #7341 / #7343)

2 participants