Skip to content

Retry import from the DockerHub on 401#8574

Merged
openshift-bot merged 1 commit intoopenshift:masterfrom
smarterclayton:retry_import
Apr 22, 2016
Merged

Retry import from the DockerHub on 401#8574
openshift-bot merged 1 commit intoopenshift:masterfrom
smarterclayton:retry_import

Conversation

@smarterclayton
Copy link
Contributor

@smarterclayton smarterclayton commented Apr 20, 2016

Follow on to #8558

Fixes #7429 #8399 #8444

@smarterclayton
Copy link
Contributor Author

[test]

@soltysh
Copy link
Contributor

soltysh commented Apr 20, 2016

LGTM, merge as soon as tests are green.

@smarterclayton
Copy link
Contributor Author

We're going to hold off on this for 1.2 just because of risk.

@smarterclayton
Copy link
Contributor Author

[test]

@smarterclayton
Copy link
Contributor Author

Flaked on #8491 [test]

On Wed, Apr 20, 2016 at 1:40 PM, OpenShift Bot notifications@github.com
wrote:

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


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#8574 (comment)

@smarterclayton
Copy link
Contributor Author

[test]

@smarterclayton
Copy link
Contributor Author

Adding a workaround for the legacy client (it's less work than ripping out
the code).

On Wed, Apr 20, 2016 at 2:46 PM, OpenShift Bot notifications@github.com
wrote:

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


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#8574 (comment)

@smarterclayton
Copy link
Contributor Author

role assignment flake

On Wed, Apr 20, 2016 at 7:00 PM, OpenShift Bot notifications@github.com
wrote:

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


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#8574 (comment)

@smarterclayton
Copy link
Contributor Author

[test]

DockerHub generates JWT tokens that are valid from "now", which means
fast actions can get a 401 unauthorized if the second boundary on the
hub servers is not aligned. We retry 401 unauthorized requests
immediately (one time), then wait a small window of time before
retrying. If our operation is too slow (longer than the window) we do
not retry.

Probably an issue with Docker Trusted Registries as well.
@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 8faa82f

@openshift-bot
Copy link
Contributor

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

@soltysh
Copy link
Contributor

soltysh commented Apr 21, 2016

Additional changes LGTM.

@soltysh soltysh added the lgtm Indicates that a PR is ready to be merged. label Apr 21, 2016
@smarterclayton
Copy link
Contributor Author

Given that flakes are blocking final merges, I'm merging this (we've had a lot of soak time). [merge]

@openshift-bot
Copy link
Contributor

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

@smarterclayton
Copy link
Contributor Author

Flake #8606 [merge]

On Fri, Apr 22, 2016 at 12:40 PM, OpenShift Bot notifications@github.com
wrote:

continuous-integration/openshift-jenkins/merge FAILURE (
https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/5660/
)


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#8574 (comment)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 8faa82f

@openshift-bot openshift-bot merged commit 4c7a595 into openshift:master Apr 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants