Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure that Chains sees all image manifests referenced #1204

Merged
merged 1 commit into from
Jul 26, 2024

Conversation

arewm
Copy link
Member

@arewm arewm commented Jul 26, 2024

Due to a bug in Chains, it does not currently see all images produced within a matrix. In order to work around this, we can expose all images that we include in our Image Index so that Chains can generate the provenance for all of them.

This fix is also relevant if any of the images added are themselves an image index due to tektoncd/chains#1070.

Before you complete this pull request ...

Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.

@arewm arewm force-pushed the ensure-all-image-manifests-signed branch 4 times, most recently from bfaaf8c to d29cf7d Compare July 26, 2024 15:50
Copy link
Member

@zregvart zregvart left a comment

Choose a reason for hiding this comment

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

LGTM

Due to a bug in Chains, it does not currently see all images produced
within a matrix. In order to work around this, we can expose all images
that we include in our Image Index so that Chains can generate the
provenance for all of them.

(and add a README for the task too)

Signed-off-by: arewm <[email protected]>
@arewm arewm force-pushed the ensure-all-image-manifests-signed branch from d29cf7d to d78b5e5 Compare July 26, 2024 15:57
@arewm arewm enabled auto-merge July 26, 2024 15:57
@arewm arewm added this pull request to the merge queue Jul 26, 2024
Merged via the queue into konflux-ci:main with commit 0de64f7 Jul 26, 2024
9 checks passed
@arewm arewm mentioned this pull request Aug 5, 2024
5 tasks
@arewm arewm deleted the ensure-all-image-manifests-signed branch November 1, 2024 17:29
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.

2 participants