Adding_suspicious_worker

[alex-rakowski]

Adds suspicious workers msg

• Tests added / passed
• Passes pre-commit run --all-files

[GPUtester]

Can one of the admins verify this patch?

Admins can comment ok to test to allow this one PR to run or add to allowlist to allow all future PRs from the same author to run.

[github-actions]

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

    29 files  ± 0      29 suites  ±0   11h 5m 59s ⏱️ + 4m 46s
 4 056 tests + 5   3 953 ✅ + 8     97 💤 ±0  6 ❌  - 3 
55 869 runs  +70  53 698 ✅ +68  2 163 💤 +4  8 ❌  - 2 

For more details on these failures, see this check.

Results for commit f4aaa34. ± Comparison against base commit e4a0545.

This pull request removes 1 and adds 6 tests. Note that renamed tests count towards both.

distributed.shuffle.tests.test_shuffle ‑ test_error_receive

distributed.http.scheduler.tests.test_scheduler_http ‑ test_prometheus_collect_task_groups
distributed.shuffle.tests.test_rechunk ‑ test_partial_rechunk_taskgroups
distributed.shuffle.tests.test_shuffle ‑ test_error_receive[P2PConsistencyError]
distributed.shuffle.tests.test_shuffle ‑ test_error_receive[ValueError]
distributed.tests.test_spans ‑ test_collections_metadata
distributed.tests.test_worker ‑ test_suppress_compute_failure_for_cancelled_tasks

♻️ This comment has been updated with latest results.

[fjetter]

add to allowlist

[alex-rakowski]

change event to action, maybe flag as an issue to enforce it/

[hendrikmakait]

Thanks, @alex-rakowski!

[hendrikmakait]

It looks like test_worker_status_sync and test_log_remove_worker aren't happy :(

Can you please adjust these? (I think you simply have to add the safe keyword and value to the respective expected outputs.)

[alex-rakowski]

Those two are now passing

[hendrikmakait]

Thanks, @alex-rakowski, and congrats on getting your first contribution merged!