Skip to content

Remove timeout in test_actor_lifetime_load_balancing.#5659

Merged
robertnishihara merged 3 commits intoray-project:masterfrom
robertnishihara:actorlifetimetest
Sep 8, 2019
Merged

Remove timeout in test_actor_lifetime_load_balancing.#5659
robertnishihara merged 3 commits intoray-project:masterfrom
robertnishihara:actorlifetimetest

Conversation

@robertnishihara
Copy link
Copy Markdown
Collaborator

@robertnishihara robertnishihara commented Sep 7, 2019

This is a follow up to #5463, which didn't fix the issue.

The change to .travis.yml needs to be reverted before merging.

@AmplabJenkins
Copy link
Copy Markdown

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/16883/
Test FAILed.

@robertnishihara
Copy link
Copy Markdown
Collaborator Author

I've confirmed that this fixes the issue. Weirdly, the test seems to always take either 5, 6, or 21 seconds...

@robertnishihara robertnishihara changed the title [DO NOT MERGE] Remove timeout in test_actor_lifetime_load_balancing. Remove timeout in test_actor_lifetime_load_balancing. Sep 8, 2019
@robertnishihara robertnishihara merged commit 87adb5a into ray-project:master Sep 8, 2019
@robertnishihara robertnishihara deleted the actorlifetimetest branch September 8, 2019 23:11
@AmplabJenkins
Copy link
Copy Markdown

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/16890/
Test PASSed.

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