Skip to content

Commit 8ef398e

Browse files
committed
Hopefully make linters happier
1 parent 5d81cd7 commit 8ef398e

File tree

4 files changed

+14
-12
lines changed

4 files changed

+14
-12
lines changed

distributed/deploy/spec.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -397,12 +397,12 @@ async def _correct_state_internal(self) -> None:
397397

398398
def _update_worker_status(self, op, msg):
399399
if op == "remove":
400-
name = msg['name']
400+
name = msg["name"]
401401

402402
def f():
403403
if (
404404
name in self.workers
405-
and msg['worker'] not in self.scheduler_info["workers"]
405+
and msg["worker"] not in self.scheduler_info["workers"]
406406
and not any(
407407
d["name"] == name
408408
for d in self.scheduler_info["workers"].values()

distributed/diagnostics/plugin.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,7 @@ def add_worker(self, scheduler: Scheduler, worker: str) -> None | Awaitable[None
190190
"""
191191

192192
def remove_worker(
193-
self, scheduler: Scheduler, worker: str, *, stimulus_id: str, **kwargs: Any
193+
self, scheduler: Scheduler, worker: str, name: str, *, stimulus_id: str, **kwargs: Any
194194
) -> None | Awaitable[None]:
195195
"""Run when a worker leaves the cluster
196196

distributed/scheduler.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9410,11 +9410,13 @@ def add_worker(self, scheduler: Scheduler, worker: str) -> None:
94109410
except CommClosedError:
94119411
scheduler.remove_plugin(name=self.name)
94129412

9413-
def remove_worker(self, scheduler: Scheduler, worker: str, name: str, **kwargs: Any) -> None:
9413+
def remove_worker(
9414+
self, scheduler: Scheduler, worker: str, name: str, **kwargs: Any
9415+
) -> None:
94149416
try:
94159417
msg = {
9416-
'worker': worker,
9417-
'name': name,
9418+
"worker": worker,
9419+
"name": name,
94189420
}
94199421
self.bcomm.send(["remove", msg])
94209422
except CommClosedError:

distributed/tests/test_stress.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,18 @@
11
from __future__ import annotations
22

33
import asyncio
4+
import copy
45
import random
56
from contextlib import suppress
67
from operator import add
78
from time import sleep
8-
import copy
99

1010
import pytest
1111
from tlz import concat, sliding_window
1212

1313
from dask import delayed
1414

15-
from distributed import Client, Nanny, Worker, wait, SpecCluster
15+
from distributed import Client, Nanny, SpecCluster, Worker, wait
1616
from distributed.chaos import KillWorker
1717
from distributed.compatibility import WINDOWS
1818
from distributed.metrics import time
@@ -342,16 +342,16 @@ async def test_chaos_rechunk(c, s, *workers):
342342

343343
@pytest.mark.slow
344344
def test_stress_scale():
345-
for n in range(30):
345+
for _ in range(30):
346346
cluster_kwargs = {}
347347
client_kwargs = {
348-
'set_as_default': False,
348+
"set_as_default": False,
349349
}
350350
spec = {}
351-
template = {'cls': Nanny}
351+
template = {"cls": Nanny}
352352
N = 24
353353
for i in range(N):
354-
w = spec[f'worker-{i}'] = copy.copy(template)
354+
w = spec[f"worker-{i}"] = copy.copy(template)
355355
cluster = SpecCluster(
356356
workers=spec,
357357
worker=template, # <- template for newly scaled up workers

0 commit comments

Comments
 (0)