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

Allow size of image to be zero when schema1 from Hub #9118

Merged
merged 1 commit into from
Jun 1, 2016

Conversation

smarterclayton
Copy link
Contributor

The DockerHub now converts schema2 image manifests pushed by Docker 1.10
or higher clients into schema1, but when it does that it drops all
v1Compatibility objects except the top layer. When it does that it
loses size info. Since we anticipate moving to schema2 soon, and the
number of schema1 images should decline, adapt the tests to flag this
error.

[merge]

The DockerHub now converts schema2 image manifests pushed by Docker 1.10
or higher clients into schema1, but when it does that it drops all
v1Compatibility objects except the top layer.  When it does that it
loses size info.  Since we anticipate moving to schema2 soon, and the
number of schema1 images should decline, adapt the tests to flag this
error.
@smarterclayton
Copy link
Contributor Author

@legionus @pweil- @soltysh please review

@openshift-bot
Copy link
Contributor

openshift-bot commented Jun 1, 2016

continuous-integration/openshift-jenkins/merge Waiting: You are in the build queue at position: 4

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to b30f322

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to b30f322

@smarterclayton
Copy link
Contributor Author

I'm going to direct merge when tests pass.

@legionus
Copy link
Contributor

legionus commented Jun 1, 2016

You forgot about TestImageStreamImportDockerHub

@smarterclayton
Copy link
Contributor Author

It is not failing for me

@legionus
Copy link
Contributor

legionus commented Jun 1, 2016

My mistake. Sorry.

@soltysh
Copy link
Contributor

soltysh commented Jun 1, 2016

LGTM

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/4307/)

@soltysh
Copy link
Contributor

soltysh commented Jun 1, 2016

[merge]

@smarterclayton smarterclayton merged commit f5bf5fd into openshift:master Jun 1, 2016
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