From 08df4fa1a635eb97f179d578791d54e7b85379ea Mon Sep 17 00:00:00 2001 From: Gabe Joseph Date: Mon, 21 Jun 2021 12:39:47 -0600 Subject: [PATCH] Disable worker dashboard by default in tests Messages like ``` /home/runner/work/distributed/distributed/distributed/node.py:160: UserWarning: Port 8787 is already in use. Perhaps you already have a cluster running? Hosting the HTTP server on port 36717 instead ``` take a a _lot_ of lines in CI. Running the worker dashboard during tests seems unnecessary? --- distributed/utils_test.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/distributed/utils_test.py b/distributed/utils_test.py index a40cf14151d..04b6ef3decb 100644 --- a/distributed/utils_test.py +++ b/distributed/utils_test.py @@ -870,7 +870,8 @@ async def test_foo(scheduler, worker1, worker2): nthreads = ncores worker_kwargs = merge( - {"memory_limit": system.MEMORY_LIMIT, "death_timeout": 10}, worker_kwargs + {"memory_limit": system.MEMORY_LIMIT, "death_timeout": 10, "dashboard": False}, + worker_kwargs, ) def _(func):