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

Wait longer for etcd startup in integration tests #5625

Merged
merged 1 commit into from
Nov 3, 2015

Conversation

smarterclayton
Copy link
Contributor

It's possible not to form a quorum in 20s. Try doubling to 40s

Fixes #5605

[test]

@liggitt
Copy link
Contributor

liggitt commented Nov 3, 2015

it's a single server... are we expecting a quorum?

@smarterclayton
Copy link
Contributor Author

It's a quorum with itself

@smarterclayton
Copy link
Contributor Author

Now with 100% more printing out wtf is going on

@smarterclayton
Copy link
Contributor Author

[test]

Also, print etcd logs if integration test dies
@smarterclayton
Copy link
Contributor Author

[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to cb974f2

@openshift-bot
Copy link
Contributor

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

@smarterclayton
Copy link
Contributor Author

[merge]

@openshift-bot
Copy link
Contributor

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

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to cb974f2

openshift-bot pushed a commit that referenced this pull request Nov 3, 2015
@openshift-bot openshift-bot merged commit 29afd72 into openshift:master Nov 3, 2015
@smarterclayton
Copy link
Contributor Author

@smarterclayton smarterclayton added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants