Skip to content

Add script for running stress tests.#3378

Merged
pcmoritz merged 6 commits intoray-project:masterfrom
robertnishihara:stresstesting
Nov 27, 2018
Merged

Add script for running stress tests.#3378
pcmoritz merged 6 commits intoray-project:masterfrom
robertnishihara:stresstesting

Conversation

@robertnishihara
Copy link
Copy Markdown
Collaborator

@robertnishihara robertnishihara commented Nov 22, 2018 •
edited
Loading

This should make it possible to run stress tests just by doing ./test/stress_tests/run_stress_tests.sh.

TODO before merging:

  • Change from 2 nodes to 200.
  • [optional] Make it easy to test different commits/branches/repos by passing in a different argument to the script.
  • Add more tests.

@robertnishihara
Copy link
Copy Markdown
Collaborator Author

robertnishihara commented Nov 22, 2018

cc @stephanie-wang @ericl @richardliaw there are no interesting tests here yet, but this is an outline of how we can do stress testing.

@AmplabJenkins
Copy link
Copy Markdown

AmplabJenkins commented Nov 22, 2018

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

@AmplabJenkins
Copy link
Copy Markdown

AmplabJenkins commented Nov 22, 2018

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

robertnishihara
robertnishihara commented Nov 22, 2018
View reviewed changes
test/stress_tests/test_dead_actors.py Outdated

import ray

ray.init()
Copy link
Copy Markdown
Collaborator Author

@robertnishihara robertnishihara Nov 22, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs to be ray.init(redis_address="localhost:6379") in order to attach to the cluster.

@AmplabJenkins
Copy link
Copy Markdown

AmplabJenkins commented Nov 22, 2018

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

@AmplabJenkins
Copy link
Copy Markdown

AmplabJenkins commented Nov 25, 2018

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

@AmplabJenkins
Copy link
Copy Markdown

AmplabJenkins commented Nov 26, 2018

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

@robertnishihara robertnishihara changed the title [WIP] Add script for running stress tests. Add script for running stress tests. Nov 27, 2018
@AmplabJenkins
Copy link
Copy Markdown

AmplabJenkins commented Nov 27, 2018

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

@AmplabJenkins
Copy link
Copy Markdown

AmplabJenkins commented Nov 27, 2018

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

@pcmoritz pcmoritz merged commit 20b8b1d into ray-project:master Nov 27, 2018
@robertnishihara robertnishihara deleted the stresstesting branch January 7, 2020 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pcmoritz pcmoritz pcmoritz approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@robertnishihara @AmplabJenkins @pcmoritz @stephanie-wang