oci-image-tool: more verbose output

[s-urbaniak]

As per https://docs.google.com/document/d/1bsi9JCnmOMSZCDh3VmDCO1m17w1aNexX09naomcpUOs/edit the oci-image-tool should have more verbose output, more concretely:

Print ...

• the digests of objects
• the version of the spec tested against
• canonical mimetypes and alternative
• the version of the validator code
• a list of tests that were ran
• a list of errors

I did implement the last point ("a list of errors") actually already, so I am wondering what additional requirements are needed here.

/cc @philips

[runcom]

@s-urbaniak If there's anything I can help with this please let me know

[s-urbaniak]

@runcom thanks for the help! Sure, just go ahead, I'll try tackle each of those in parallel with other rkt work. Just let me know if you want to tackle all of them or just a subset. Seem like low-hanging fruits to me.

[coolljt0725]

@runcom If you just tackle a subnet of them, I'd like to help with the rest

[xiekeyang]

@s-urbaniak If it need to be done based on #159 ?
It refactor the image tool completely. If we did before it is merged, we may be meet a big conflict.

[wking]

On Tue, Aug 30, 2016 at 07:11:01PM -0700, xiekeyang wrote:

@s-urbaniak If it need to be done based on #159 ?
It refactor the image tool completely. If we did before it is
merged, we may be meet a big conflict.

I'm all for avoiding conflicts, but #159 tries to stay away from the
existing image-validation logic. So I expect you can wade into these issues without butting heads with #159.

My long-term plan was to land #159 and then rewrite the
validation/unpacking logic to use callback registries. Something
like:

type Validator func(ctx context.Context, engine cas.Engine, ref specs.Descriptor, recursive bool) (errors []string)

var Validaters map[string]Validator {
"application/vnd.oci.descriptor.v1+json": ValidateDescriptor,
"application/vnd.oci.image.manifest.v1+json": ValidateManifest,
"application/vnd.oci.image.manifest.list.v1+json": ValidateManifestList,
…
}

But I don't have any work in progress on that front, so there won't be
any conflicts if validation shifts around in the meantime ;).

[philips]

This issue was moved to opencontainers/image-tools#22