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

manifest: List media types supported by descriptor references #304

Merged
merged 1 commit into from
Sep 24, 2016

Conversation

wking
Copy link
Contributor

@wking wking commented Sep 13, 2016

Require implementations to support these media types (although an implementation which also supports additional media types is still compliant). Suggest portable manifests and manifest lists stick to these types so the can rely on implementation support.

A manifest(-list) can safely use a type from outside the list if the author expects the implementation to support the extension type (e.g. application/vnd.docker.image.rootfs.diff.tar.gzip). But how that out-of-band expectation is setup is beyond the scope of this specification.

Previous discussion in #286 and a few other one-off comments ;).

@vbatts
Copy link
Member

vbatts commented Sep 13, 2016

This seems to go against the compatibility matrix

@philips
Copy link
Contributor

philips commented Sep 13, 2016

Yes, please add the nondistributable to the compat matrix. Thanks!

@wking wking mentioned this pull request Sep 13, 2016
@wking
Copy link
Contributor Author

wking commented Sep 13, 2016

On Tue, Sep 13, 2016 at 12:12:19PM -0700, Brandon Philips wrote:

Yes, please add the nondistributable to the compat matrix. Thanks!

Should be covered by the just-landed #233. I'll rebase around that
shortly.

@wking
Copy link
Contributor Author

wking commented Sep 13, 2016

On Tue, Sep 13, 2016 at 12:00:57PM -0700, Vincent Batts wrote:

This seems to go against the compatibility matrix

Maybe I'm misunderstanding the compatibility matrix. I had thought it
was an informative hint to image translators and folks writing
implementations that wanted to support related types. Are you saying
it is normative, and that complaint OCI implementations MUST support
those non-OCI types as well? Do the other types even have versioned
specifications that we would use to certify compliance with them?

wking added a commit to wking/image-spec that referenced this pull request Sep 14, 2016
Linking the files from Makefile's DOC_FILES in their listed order.
Link text generally follows the headers used by the linked document.
Informative stuff (for image and implementation authors) is indented
under the README (although I'm not sure there's consensus on whether
media-types.md is normative or informative [1]).

[1]: opencontainers#304 (comment)
     Subject: manifest: List media types supported by descriptor references

Signed-off-by: W. Trevor King <wking@tremily.us>
@jonboulle
Copy link
Contributor

I now have similar confusion to #304 (comment) so a clarification would be appreciated.

@wking
Copy link
Contributor Author

wking commented Sep 14, 2016

On Tue, Sep 13, 2016 at 12:22:54PM -0700, W. Trevor King wrote:

Tue, Sep 13, 2016 at 12:12:19PM -0700, Brandon Philips:

Yes, please add the nondistributable to the compat matrix. Thanks!

Should be covered by the just-landed #233. I'll rebase around that
shortly.

Rebased onto master (no conflicts) with ae138eba3b9724.

@wking wking mentioned this pull request Sep 14, 2016
wking added a commit to wking/image-spec that referenced this pull request Sep 15, 2016
Linking the files from Makefile's DOC_FILES in their listed order.
Link text generally follows the headers used by the linked document.
Informative stuff (for image and implementation authors) is indented
under the README (although I'm not sure there's consensus on whether
media-types.md is normative or informative [1]).

[1]: opencontainers#304 (comment)
     Subject: manifest: List media types supported by descriptor references

Signed-off-by: W. Trevor King <wking@tremily.us>
@jonboulle
Copy link
Contributor

This patch LGTM based on my understanding so would love to hear what I'm missing..

@philips
Copy link
Contributor

philips commented Sep 21, 2016

LGTM

Approved with PullApprove

1 similar comment
@vbatts
Copy link
Member

vbatts commented Sep 22, 2016

LGTM

Approved with PullApprove

@vbatts
Copy link
Member

vbatts commented Sep 22, 2016

gd... now it needs a rebase? how did github not show that right before? ...

rebase please

Require implementations to support these media types (although an
implementation which also supports additional media types is still
compliant).  Suggest portable manifests and manifest lists stick to
these types so the can rely on implementation support.

A manifest(-list) can safely use a type from outside the list if the
author expects the implementation to support the extension type
(e.g. application/vnd.docker.image.rootfs.diff.tar.gzip).  But how
that out-of-band expectation is setup is beyond the scope of this
specification.

Signed-off-by: W. Trevor King <wking@tremily.us>
@wking
Copy link
Contributor Author

wking commented Sep 22, 2016

On Thu, Sep 22, 2016 at 10:49:12AM -0700, Vincent Batts wrote:

rebase please

Rebased around #305 with a3b9724b1f1d9c.

@vbatts
Copy link
Member

vbatts commented Sep 22, 2016

LGTM

Approved with PullApprove

@jonboulle
Copy link
Contributor

jonboulle commented Sep 24, 2016

lgtm

Approved with PullApprove

@jonboulle jonboulle merged commit 8f4f36b into opencontainers:master Sep 24, 2016
@wking wking deleted the media-type-requirements branch September 29, 2016 17:43
wking added a commit to wking/image-spec that referenced this pull request Oct 16, 2016
I typo'd these in b1f1d9c (manifest: List media types supported by
descriptor references, 2016-09-13, opencontainers#304).

Signed-off-by: W. Trevor King <wking@tremily.us>
dattgoswami9lk5g added a commit to dattgoswami9lk5g/bremlinr that referenced this pull request Jun 6, 2022
Linking the files from Makefile's DOC_FILES in their listed order.
Link text generally follows the headers used by the linked document.
Informative stuff (for image and implementation authors) is indented
under the README (although I'm not sure there's consensus on whether
media-types.md is normative or informative [1]).

[1]: opencontainers/image-spec#304 (comment)
     Subject: manifest: List media types supported by descriptor references

Signed-off-by: W. Trevor King <wking@tremily.us>
7c00d pushed a commit to 7c00d/J1nHyeockKim that referenced this pull request Jun 6, 2022
Linking the files from Makefile's DOC_FILES in their listed order.
Link text generally follows the headers used by the linked document.
Informative stuff (for image and implementation authors) is indented
under the README (although I'm not sure there's consensus on whether
media-types.md is normative or informative [1]).

[1]: opencontainers/image-spec#304 (comment)
     Subject: manifest: List media types supported by descriptor references

Signed-off-by: W. Trevor King <wking@tremily.us>
7c00d added a commit to 7c00d/J1nHyeockKim that referenced this pull request Jun 6, 2022
Linking the files from Makefile's DOC_FILES in their listed order.
Link text generally follows the headers used by the linked document.
Informative stuff (for image and implementation authors) is indented
under the README (although I'm not sure there's consensus on whether
media-types.md is normative or informative [1]).

[1]: opencontainers/image-spec#304 (comment)
     Subject: manifest: List media types supported by descriptor references

Signed-off-by: W. Trevor King <wking@tremily.us>
laventuraw added a commit to laventuraw/Kihara-tony0 that referenced this pull request Jun 6, 2022
Linking the files from Makefile's DOC_FILES in their listed order.
Link text generally follows the headers used by the linked document.
Informative stuff (for image and implementation authors) is indented
under the README (although I'm not sure there's consensus on whether
media-types.md is normative or informative [1]).

[1]: opencontainers/image-spec#304 (comment)
     Subject: manifest: List media types supported by descriptor references

Signed-off-by: W. Trevor King <wking@tremily.us>
tomalopbsr0tt added a commit to tomalopbsr0tt/fabiojosej that referenced this pull request Oct 6, 2022
Linking the files from Makefile's DOC_FILES in their listed order.
Link text generally follows the headers used by the linked document.
Informative stuff (for image and implementation authors) is indented
under the README (although I'm not sure there's consensus on whether
media-types.md is normative or informative [1]).

[1]: opencontainers/image-spec#304 (comment)
     Subject: manifest: List media types supported by descriptor references

Signed-off-by: W. Trevor King <wking@tremily.us>
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.

4 participants