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

Support ManifestList in dockerregistry and importer #16323

Closed
2 tasks
soltysh opened this issue Sep 13, 2017 · 2 comments
Closed
2 tasks

Support ManifestList in dockerregistry and importer #16323

soltysh opened this issue Sep 13, 2017 · 2 comments
Assignees
Milestone

Comments

@soltysh
Copy link
Member

soltysh commented Sep 13, 2017

Dockerhub serves ManifestList instead of regular Manifest if you provide it with information about having that type supported. Whenever we import github.com/docker/distribution/manifest/manifestlist which we do indirectly in integration tests we tell docherhub we do support this.

There was one attempt to bypass the problem temporarily: #16319 but that failed.

@dmage is currently working on a permanent fix

Make sure to revert:

@soltysh
Copy link
Member Author

soltysh commented Sep 14, 2017

I've added links to prs/commits that should be reverted when this will be solve.

@mfojtik mfojtik assigned bparees and unassigned mfojtik Sep 14, 2017
@bparees
Copy link
Contributor

bparees commented Sep 21, 2017

fixed by #16322

all relevant PRs were reverted(ie tests re-enabled) with that work as well.

@bparees bparees closed this as completed Sep 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants