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

UPSTREAM: 14182: Distinguish image registry unavailable and pull failure #5653

Merged
merged 1 commit into from Nov 4, 2015

Conversation

derekwaynecarr
Copy link
Member

This is a prerequisite to serial image pulling in Kubelet.

@smarterclayton @liggitt

@smarterclayton
Copy link
Contributor

LGTM does it also have tests upstream?

@smarterclayton
Copy link
Contributor

p1 due to serial pull being on the blocker list.

@smarterclayton smarterclayton added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 3, 2015
@pweil- pweil- mentioned this pull request Nov 3, 2015
@eparis
Copy link
Member

eparis commented Nov 3, 2015

LGTM

@eparis eparis added the lgtm Indicates that a PR is ready to be merged. label Nov 3, 2015
@smarterclayton smarterclayton removed the lgtm Indicates that a PR is ready to be merged. label Nov 4, 2015
@smarterclayton
Copy link
Contributor

[merge]

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/6861/) (Image: devenv-rhel7_2638)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 8f4c24b

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 8f4c24b

@openshift-bot
Copy link
Contributor

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

openshift-bot pushed a commit that referenced this pull request Nov 4, 2015
@openshift-bot openshift-bot merged commit efaeba0 into openshift:master Nov 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. priority/P1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants