Skip to content

[release/2.0] Update image type checks to avoid unnecessary logs for attestations#11537

Merged
AkihiroSuda merged 2 commits intocontainerd:release/2.0from
k8s-infra-cherrypick-robot:cherry-pick-11327-to-release/2.0
Mar 14, 2025
Merged

[release/2.0] Update image type checks to avoid unnecessary logs for attestations#11537
AkihiroSuda merged 2 commits intocontainerd:release/2.0from
k8s-infra-cherrypick-robot:cherry-pick-11327-to-release/2.0

Conversation

@k8s-infra-cherrypick-robot

This is an automated cherry-pick of #11327

/assign dmcgowan

vvoland added 2 commits March 13, 2025 18:18
Before this patch, calling `image.Children` on an image built with
BuildKit would produce unnecessary `encountered unknown type
application/vnd.in-toto+json; children may not be fetched` debug logs,
because the media type is neither a known layer or config type.

Make the `image.Children` aware of the attestation layers and don't
attempt to traverse them.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
Don't produce `reference for unknown type: application/vnd.in-toto+json`
warning logs when pushing/fetching an image containing the attestation
manifests.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
@k8s-ci-robot
Copy link

Hi @k8s-infra-cherrypick-robot. Thanks for your PR.

I'm waiting for a containerd member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@dmcgowan dmcgowan changed the title [release/2.0] Don't produce unnecessary logs when encountering attestations [release/2.0] Update image type checks to avoid unnecessary logs for attestations Mar 13, 2025
@djdongjin
Copy link
Member

/ok-to-test

@github-project-automation github-project-automation bot moved this from Needs Triage to Review In Progress in Pull Request Review Mar 14, 2025
@AkihiroSuda AkihiroSuda merged commit b6ab437 into containerd:release/2.0 Mar 14, 2025
58 checks passed
@github-project-automation github-project-automation bot moved this from Review In Progress to Done in Pull Request Review Mar 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants