Skip to content

Commit

Permalink
Increate timeouts of tests that frequently timeout in CI (#1228)
Browse files Browse the repository at this point in the history
In the past few weeks some tests have timed out with certain frequency in CI, probably due to its load. Attempt to avoid those by increasing timeouts from 20 to 30 seconds.

Authors:
  - Peter Andreas Entschev (https://github.com/pentschev)

Approvers:
  - Ray Douglass (https://github.com/raydouglass)
  - Benjamin Zaitlen (https://github.com/quasiben)

URL: #1228
  • Loading branch information
pentschev authored Aug 29, 2023
1 parent 1c7b1ce commit 171fd2c
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 2 deletions.
1 change: 1 addition & 0 deletions ci/test_python.sh
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@ UCX_WARN_UNUSED_ENV_VARS=n \
UCX_MEMTYPE_CACHE=n \
timeout 40m pytest \
-vv \
--durations=0 \
--capture=no \
--cache-clear \
--junitxml="${RAPIDS_TESTS_DIR}/junit-dask-cuda.xml" \
Expand Down
1 change: 1 addition & 0 deletions dask_cuda/tests/test_local_cuda_cluster.py
Original file line number Diff line number Diff line change
Expand Up @@ -454,6 +454,7 @@ async def test_get_cluster_configuration():
@gen_test(timeout=20)
async def test_worker_fraction_limits():
async with LocalCUDACluster(
dashboard_address=None,
device_memory_limit=0.1,
rmm_pool_size=0.2,
rmm_maximum_pool_size=0.3,
Expand Down
2 changes: 1 addition & 1 deletion dask_cuda/tests/test_proxify_host_file.py
Original file line number Diff line number Diff line change
Expand Up @@ -384,7 +384,7 @@ def test_incompatible_types(root_dir):

@pytest.mark.parametrize("npartitions", [1, 2, 3])
@pytest.mark.parametrize("compatibility_mode", [True, False])
@gen_test(timeout=20)
@gen_test(timeout=30)
async def test_compatibility_mode_dataframe_shuffle(compatibility_mode, npartitions):
cudf = pytest.importorskip("cudf")

Expand Down
2 changes: 1 addition & 1 deletion dask_cuda/tests/test_proxy.py
Original file line number Diff line number Diff line change
Expand Up @@ -400,7 +400,7 @@ def _pxy_deserialize(self):

@pytest.mark.parametrize("send_serializers", [None, ("dask", "pickle"), ("cuda",)])
@pytest.mark.parametrize("protocol", ["tcp", "ucx"])
@gen_test(timeout=20)
@gen_test(timeout=60)
async def test_communicating_proxy_objects(protocol, send_serializers):
"""Testing serialization of cuDF dataframe when communicating"""
cudf = pytest.importorskip("cudf")
Expand Down

0 comments on commit 171fd2c

Please sign in to comment.