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

Add OCI artifact support, test syncing Cosign signatures again #1680

Merged

Conversation

mtrmac
Copy link
Collaborator

@mtrmac mtrmac commented Jun 16, 2022

@mtrmac mtrmac force-pushed the test-registry-2.8.1-with-cosign-signatures branch from 2ea01cf to 428b19b Compare June 16, 2022 18:17
@mtrmac mtrmac force-pushed the test-registry-2.8.1-with-cosign-signatures branch 2 times, most recently from fce9cad to dcbd96d Compare June 17, 2022 19:16
@mtrmac
Copy link
Collaborator Author

mtrmac commented Jun 17, 2022

This now vendors c/image from the main branch again. From a Skopeo point of view it could be merged, but we need containers/image#1581 first so that the c/image CI doesn’t break.

@mtrmac mtrmac force-pushed the test-registry-2.8.1-with-cosign-signatures branch from dcbd96d to d021055 Compare June 21, 2022 17:48
@mtrmac mtrmac changed the title DO NOT MERGE: Add OCI artifact support, test syncing Cosign signatures again Add OCI artifact support, test syncing Cosign signatures again Jun 21, 2022
@mtrmac
Copy link
Collaborator Author

mtrmac commented Jun 21, 2022

Now mergeable. Note that it still vendors an unreleased c/image.

@mtrmac mtrmac marked this pull request as ready for review June 21, 2022 17:49
@mtrmac mtrmac force-pushed the test-registry-2.8.1-with-cosign-signatures branch 2 times, most recently from cb3b949 to 9a2c49d Compare June 25, 2022 13:05
@mtrmac mtrmac force-pushed the test-registry-2.8.1-with-cosign-signatures branch from 9a2c49d to e8da053 Compare June 29, 2022 17:40
@rhatdan
Copy link
Member

rhatdan commented Jul 1, 2022

@vrothberg PTAL

Copy link
Member

@vrothberg vrothberg left a comment

Choose a reason for hiding this comment

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

LGTM

@vrothberg
Copy link
Member

/lgtm

@vrothberg
Copy link
Member

Ah, no bot here.

@mtrmac, could you rebase before we force merge?

including containers/image#1574 .

> go get github.com/containers/image/v5@main
> make vendor

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
This reverts commit bbdabeb.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
@mtrmac mtrmac force-pushed the test-registry-2.8.1-with-cosign-signatures branch from e8da053 to 61593fc Compare July 1, 2022 15:35
@mtrmac
Copy link
Collaborator Author

mtrmac commented Jul 1, 2022

Rebased.

@vrothberg vrothberg merged commit b95e081 into containers:main Jul 4, 2022
@mtrmac mtrmac deleted the test-registry-2.8.1-with-cosign-signatures branch July 4, 2022 14:51
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants