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

OCI image-spec / distribution-spec v1.1 updates, first round #2062

Merged
merged 13 commits into from
Aug 10, 2023

Conversation

mtrmac
Copy link
Collaborator

@mtrmac mtrmac commented Jul 31, 2023

This is an assorted set of small changes to support OCI image-spec/distribution-spec 1.1.

Follow #2030 for checklists of things to update/address.

See individual commit messages for details. Note especially the RFC and UNTESTED ones.

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

@mtrmac mtrmac added the kind/feature A request for, or a PR adding, new functionality label Aug 4, 2023
@mtrmac mtrmac force-pushed the oci-1.1-little branch 2 times, most recently from 3cfb544 to 5a70811 Compare August 7, 2023 19:52
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
... so update one of our checks to rely on the specified status (only).

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
... as now requested by distribution-spec.

Untested apart from the added parser unit test.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
As described in the comment, this might cause excess noise.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
... to reduce duplication.

Should not change behavior.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Should not change behavior.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
... so that newOCI1ImageSource can support other configs.

Should not change (test) behavior.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Should not change (test) behavior for now.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
This is a MUST requirement from OCI image spec v1.1.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Zero-layer artifacts are discouraged but allowed.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
@rhatdan
Copy link
Member

rhatdan commented Aug 10, 2023

LGTM

@rhatdan rhatdan merged commit e626be2 into containers:main Aug 10, 2023
9 checks passed
@mtrmac mtrmac deleted the oci-1.1-little branch August 10, 2023 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature A request for, or a PR adding, new functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants