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

Proposal J - new media type for compatibilities with update in image … #20

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ChaoyiHuang
Copy link
Collaborator

new media type for compatibilities with update in image index platform

This proposal is born from proposal A to I, providing another option to hold compatibilities, and retriving the compatibilities in shorter path.

The purpose of the proposal, based on the maintainers feedback, is to address the concern maintaining a standalone artifact specification in OCI.

…index platform

This proposal is born from proposal A to I, providing another option
to hold compatibilities, and retriving the compatibilities in shorter path.

The purpose of the proposal, based on the maintainers feedback, is to
address the concern maintaining a standalone artifact specification in OCI.

Signed-off-by: joehuang <[email protected]>
Copy link
Collaborator

@sudo-bmitch sudo-bmitch left a comment

Choose a reason for hiding this comment

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

Putting this on hold until we can discuss the risk of unexpected data loss.


### add platform.compat descriptor

Add a new descriptor in platform object in [image index](https://github.com/opencontainers/image-spec/blob/main/image-index.md#image-index-property-descriptions).
Copy link
Collaborator

Choose a reason for hiding this comment

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

I have concerns that this descriptor would not be known to existing registries and client tooling. As a result, registries would accept an incomplete manifest, or silently delete the compatibility blob in a later GC cycle. And client tooling to copy images would fail to include the compatibility blobs.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yes, this is the risk. some experiments need to be done to investigate the result of unknown descriptor

Copy link
Collaborator

@mfranczy mfranczy Jun 17, 2024

Choose a reason for hiding this comment

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

@sudo-bmitch is right, so far I confirmed that with regclient and zot registry.
For this proposal we would have to add a descriptor and a new media type support to regclient and zot.
Didn't test other registries and tools yet.

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.

3 participants