Skip to content

deploy: revert proportional recreate timeout and default to 10m#12259

Merged
openshift-bot merged 1 commit intoopenshift:masterfrom
mfojtik:revert-proportional-recreate
Dec 16, 2016
Merged

deploy: revert proportional recreate timeout and default to 10m#12259
openshift-bot merged 1 commit intoopenshift:masterfrom
mfojtik:revert-proportional-recreate

Conversation

@mfojtik
Copy link
Contributor

@mfojtik mfojtik commented Dec 14, 2016

This PR reverts the proportional timeout (accounting the deployer pod start time) as we don't need that since we count timeout for each operation that makes progress. Also this bumps the default timeout for this in recreate from 2m to 10m.

@mfojtik mfojtik requested a review from 0xmichalis December 14, 2016 13:53
@mfojtik
Copy link
Contributor Author

mfojtik commented Dec 14, 2016

[test]

@Kargakis PTAL

@0xmichalis
Copy link
Contributor

@mfojtik thanks, lgtm [merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to fb84a4c

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/12376/) (Base Commit: ec9ff39)

@mfojtik
Copy link
Contributor Author

mfojtik commented Dec 15, 2016

flake: #10228

[merge]

@mfojtik
Copy link
Contributor Author

mfojtik commented Dec 16, 2016

flake: #12184

[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to fb84a4c

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 16, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/12448/) (Base Commit: 4cb1ce3) (Image: devenv-rhel7_5555)

@openshift-bot openshift-bot merged commit 8e01869 into openshift:master Dec 16, 2016
@mfojtik mfojtik deleted the revert-proportional-recreate branch September 5, 2018 21:08
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