Skip to content
This repository has been archived by the owner on Jul 2, 2020. It is now read-only.

Support HEAD /v2/<team>/<artifact>/manifests/<name> #160

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

Conversation

mikkeloscar
Copy link
Member

@mikkeloscar mikkeloscar commented Jun 7, 2018

This adds support for responding to HEAD /v2/<team>/<artifact>/manifests/<name> requests.
This is required by containerd and described in the opencontainers spec: https://github.com/opencontainers/distribution-spec/blob/master/spec.md#existing-manifests

Reference: containerd/cri#787

I need this to pull skipper in my own Kubernetes cluster running containerd instead of docker :P

@hjacobs
Copy link
Contributor

hjacobs commented Jun 7, 2018

@mikkeloscar @dryewo has some "project" going on which would make your change obsolete.. 🙊 🤔

@dryewo
Copy link
Contributor

dryewo commented Jun 7, 2018

Let's discuss it tomorrow. I'm sure it would take maybe 15 minutes to apply Mikkel's patch to the modern implementation of Pier One.

Adds support for responding to `HEAD /v2/<team>/<artifact>/manifests/<name>` requests.
This is required by containerd and described in the opencontainers spec: https://github.com/opencontainers/distribution-spec/blob/master/spec.md

Signed-off-by: Mikkel Oscar Lyderik Larsen <m@moscar.net>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants