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

node-controller: deflake TestUpdateNodeWithMultiplePods #13940

Merged

Conversation

sttts
Copy link
Contributor

@sttts sttts commented Apr 28, 2017

Fixes #13941

Use absolute time and increase grace duration to 500ms for tests under load.

@sttts sttts force-pushed the sttts-deflake-TestUpdateNodeWithMultiplePods branch 3 times, most recently from c2b63d4 to 793292c Compare April 28, 2017 11:59
Use absolute time and increase grace duration to 500ms for tests under load.
@sttts sttts force-pushed the sttts-deflake-TestUpdateNodeWithMultiplePods branch from 793292c to 90e8539 Compare April 28, 2017 13:18
@sttts
Copy link
Contributor Author

sttts commented Apr 28, 2017

[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 90e8539

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test Running (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin/1030/) (Base Commit: 1596f74)

@deads2k
Copy link
Contributor

deads2k commented Apr 28, 2017

lgtm [merge]

@openshift-bot
Copy link
Contributor

openshift-bot commented Apr 28, 2017

continuous-integration/openshift-jenkins/merge Waiting: You are in the build queue at position: 14

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 90e8539

@deads2k
Copy link
Contributor

deads2k commented Apr 28, 2017

Flake fix that only touches a unit test. Test went green. Merging.

@deads2k deads2k merged commit cd4661d into openshift:master Apr 28, 2017
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