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

Clarify image index usage with same-platform manifests #776

Closed
hendrikhalkow opened this issue Apr 28, 2019 · 1 comment · Fixed by #880
Closed

Clarify image index usage with same-platform manifests #776

hendrikhalkow opened this issue Apr 28, 2019 · 1 comment · Fixed by #880

Comments

@hendrikhalkow
Copy link

I think the image index needs a bit more clarification on how an image is intended to be processed by a runtime spec when there are multiple manifests within one image. While it is clear what is meant when there is exactly one manifest within an image or two manifests for different platform or architectures within an image, it is unclear when there are manifests for the same platform and architecture within an image. As of now, it is not forbidden.

Is it intended that there is no restriction of having multiple manifests for the same architecture and platform within an image?

Or, is it up to the runtime-spec implementation on how to handle situations like this, e.g. by adding some annotation-based selector like

podman run --select org.opencontainers.image.ref.name=myref myimage:latest

or a sub-image dressing scheme like podman run myimage:latest/myref ?

Even if this is intentional, there should be a few more sentences in the image index spec on how to handle this.

@jonjohnsonjr
Copy link
Contributor

jonjohnsonjr commented May 1, 2019

There is a lot of discussion in #581 that is relevant. It's a long one, but I believe the general consensus is that it's up to the implementation to decide if it should return an error or just pick an arbitrary entry in the face of ambiguity, e.g. this algorithm "looks about right".

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 a pull request may close this issue.

2 participants