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: 23746: A pod never terminated if a container image registry was unavailable #8378

Merged
merged 1 commit into from
Apr 12, 2016

Conversation

ncdc
Copy link
Contributor

@ncdc ncdc commented Apr 6, 2016

@derekwaynecarr derekwaynecarr mentioned this pull request Apr 6, 2016
85 tasks
@ncdc
Copy link
Contributor Author

ncdc commented Apr 6, 2016

[test]

1 similar comment
@ncdc
Copy link
Contributor Author

ncdc commented Apr 6, 2016

[test]

@derekwaynecarr
Copy link
Member

LGTM , thanks @ncdc for getting this cherry pick.

@ncdc
Copy link
Contributor Author

ncdc commented Apr 6, 2016

[test]

@ncdc
Copy link
Contributor Author

ncdc commented Apr 6, 2016

@derekwaynecarr I'm delegating to you to babysit the tests here :p

@ncdc
Copy link
Contributor Author

ncdc commented Apr 7, 2016

[test]

@ncdc
Copy link
Contributor Author

ncdc commented Apr 7, 2016

Hit kubernetes/kubernetes#22639
[test]

@derekwaynecarr
Copy link
Member

[test]

2 similar comments
@smarterclayton
Copy link
Contributor

[test]

@ncdc
Copy link
Contributor Author

ncdc commented Apr 8, 2016

[test]

@ncdc
Copy link
Contributor Author

ncdc commented Apr 8, 2016

--- FAIL: TestNewAppRunBuilds-2 (2.21s)
    newapp_test.go:1198: successful input image source build with no repository: unexpected error: no match for "centos/mongodb-26-centos7"

[test]

@ncdc
Copy link
Contributor Author

ncdc commented Apr 8, 2016

Summarizing 3 Failures:

[Fail] deployments: parallel: test deployment test deployment [It] should run a deployment to completion and then scale to zero 
/data/src/github.com/openshift/origin/test/extended/deployments/deployments.go:29

[Fail] [job] openshift can execute jobs controller [It] should create and run a job in user project 
/data/src/github.com/openshift/origin/test/extended/jobs/jobs.go:26

[Fail] Kubectl client Update Demo [It] should scale a replication controller [Conformance] 
/data/src/github.com/openshift/origin/Godeps/_workspace/src/k8s.io/kubernetes/test/e2e/util.go:1469

Ran 118 of 383 Specs in 1327.214 seconds
FAIL! -- 115 Passed | 3 Failed | 0 Pending | 265 Skipped 

[test]

@smarterclayton smarterclayton added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 11, 2016
@smarterclayton
Copy link
Contributor

Approved

@ncdc
Copy link
Contributor Author

ncdc commented Apr 12, 2016

[test]

@ncdc
Copy link
Contributor Author

ncdc commented Apr 12, 2016

Conformance test failed due to an unrelated networking issue:

Cannot find a valid baseurl for repo: extras/7/x86_64
Could not retrieve mirrorlist http://mirrorlist.centos.org/?release=7&arch=x86_64&repo=extras&infra=stock error was
12: Timeout on http://mirrorlist.centos.org/?release=7&arch=x86_64&repo=extras&infra=stock: (28, 'Operation too slow. Less than 1000 bytes/sec transferred the last 30 seconds')

@smarterclayton
Copy link
Contributor

[merge]

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/2941/) (Image: devenv-rhel7_3957)

@ncdc
Copy link
Contributor Author

ncdc commented Apr 12, 2016

[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 984dde8

@smarterclayton
Copy link
Contributor

font awesome broken again @jwforres

On Tue, Apr 12, 2016 at 3:55 PM, OpenShift Bot [email protected]
wrote:

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


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#8378 (comment)

@ncdc
Copy link
Contributor Author

ncdc commented Apr 12, 2016

[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 984dde8

@liggitt
Copy link
Contributor

liggitt commented Apr 12, 2016

fontawesome fix merged

@openshift-bot
Copy link
Contributor

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

@openshift-bot openshift-bot merged commit 5986a92 into openshift:master Apr 12, 2016
@mdshuai
Copy link

mdshuai commented Apr 14, 2016

@ncdc How to make the pod status to "RegistryUnavailable" ? I use a unavailable registry for a pod, but it can't return this error "the registry is unavailable".

@ncdc
Copy link
Contributor Author

ncdc commented Apr 14, 2016

@mdshuai it was @derekwaynecarr who did all the hard work researching this. Derek, do you have any suggestions for QE?

@derekwaynecarr
Copy link
Member

the error would only occur if docker itself returned a 50x error. I do not know a reliable way to force docker to return this as using an unavailable registry would have been a 40x error.

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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants