-
Notifications
You must be signed in to change notification settings - Fork 82
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
Validation strictness #66
Comments
@wking |
A use case for unpacking a single layer? Maybe unpacking a manifest is not giving you the results you expect, and you want to step through the layers one by one. |
This was referenced Feb 3, 2017
This was referenced Feb 15, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Refs can point to whichever type they like (although they usually point at manifests or manifest lists). How should we handle validation when a ref points at a single layer? Or at a config? Or at an image/png? I see a few interesting settings:
--unpackable
, for “can image-tools unpack this content?”. This would be true for manifests, manifest lists, and (once we land support for it) layers, but would not be true for configs (because diffIDs are not neccessarily fetchable from CAS).--strict
, for “does the validated object only contain content which the OCI defines (or references) in the spec?”. In that case, refs pointing to manifests, manifest lists, configs, or layers would allbe legal. Other types would trigger validation errors.
--base
, for “does the validated object only contain content which the OCI requires unpackers to support?”. This would be true for manifests and layers, but not for manifest lists, configs, or othertypes. There has been talk about requiring unpacker support for manifest lists, we currently only SHOULD support for them. If we MUST support for them, then there may not be any need for
--base
(since it would cover the same types as--strict
).In cases where the user specified none of
--unpackable
,--strict
, or--base
, I'd rather warn (but not error) on this sort of thing. And using TAP (as described in #60) gives us a convenient way to warnwithout erroring.
The text was updated successfully, but these errors were encountered: