Skip to content

Comments

Check index.docker.io/v1 when auth.docker.io/token has no auth#7047

Merged
openshift-bot merged 1 commit intoopenshift:masterfrom
smarterclayton:fix_dockerhub_import
Feb 5, 2016
Merged

Check index.docker.io/v1 when auth.docker.io/token has no auth#7047
openshift-bot merged 1 commit intoopenshift:masterfrom
smarterclayton:fix_dockerhub_import

Conversation

@smarterclayton
Copy link
Contributor

The Docker daemon will check an older auth store value for challenges,
so we have to as well.

Fixed a bug where image stream import wasn't setting spec tags (which
meant oc import-image couldn't be invoked again if the first time
failed) and where the conditions weren't getting cleared if a second
import ran successfully with the same output image.

Fixes Bugz 1304239

The Docker daemon will check an older auth store value for challenges,
so we have to as well.

Fixed a bug where image stream import wasn't setting spec tags (which
meant oc import-image couldn't be invoked again if the first time
failed) and where the conditions weren't getting cleared if a second
import ran successfully with the same output image.
@soltysh
Copy link
Contributor

soltysh commented Feb 5, 2016

LGTM [merge]

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/4860/) (Image: devenv-rhel7_3341)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to b269962

openshift-bot pushed a commit that referenced this pull request Feb 5, 2016
@openshift-bot openshift-bot merged commit d7b3f51 into openshift:master Feb 5, 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.

3 participants