[travis] Add back '-v' option to pytest and install psutil #4430
Conversation
|
Can one of the admins verify this patch? |
.travis.yml
Outdated
| - python -m pytest -v --durations=5 python/ray/tests/test_mini.py | ||
| - python -m pytest -v --durations=5 python/ray/tests/test_array.py | ||
| - python -m pytest -v --durations=5 python/ray/tests/test_multi_node_2.py | ||
| - python -m pytest -v --durations=5 python/ray/tests/test_node_manager.py |
There was a problem hiding this comment.
Can we wrap these in ci/suppress_output then?
It will hide the durations though, but will print output if the test hangs.
I think there is also a way to set a global timeout in pytest: https://pypi.org/project/pytest-timeout/1.2.1/
There was a problem hiding this comment.
ci/suppress_output will also hide the progress. Considering -v only adds 450-ish lines, I think that is fine?
And the global timeout is a good idea.
There was a problem hiding this comment.
Ok, let's see how it goes. I think this might be ok (mobile browsers seems to start choking only at a couple thousand lines).
|
Test FAILed. |
This reverts commit f287249.
|
Test PASSed. |
|
Test FAILed. |
What do these changes do?
-voption. So when a test hangs, we know which test it is.Related issue number