Conversation
e3e1330 to
83ba052
Compare
Unit Test ResultsSee test report for an extended history of previous test failures. This is useful for diagnosing flaky tests. 0 tests ±0 0 ✅ ±0 0s ⏱️ ±0s Results for commit 59dab32. ± Comparison against base commit 0a07553. ♻️ This comment has been updated with latest results. |
23137f9 to
8c314d3
Compare
| # Check cgroups if available | ||
| if sys.platform == "linux": | ||
| if LINUX: | ||
| quota, period = _try_extract_cgroup_cpu_quota() |
There was a problem hiding this comment.
Out of scope: this should belong to os.process_cpu_count. As it is complicated to test without mocking, I did not investigate if any of it is actually implemented. Let's revisit this after Python 3.13 becomes the minimum version in a few years.
|
|
||
| from dask.system import cpu_count | ||
|
|
||
| psutil = pytest.importorskip("psutil") |
There was a problem hiding this comment.
Run most tests when psutil is not installed
This reverts commit 71ff2c2.
|
@jacobtomlinson @jsignell does either of you have bandwidth to offer review? |
dcherian
left a comment
There was a problem hiding this comment.
seems fine from a quick look
int | NonetointThis was tested on an enhanced CI matrix (71ff2c2) and returned all green (https://github.com/crusaderky/dask/actions/runs/20781477335/job/59679809102). CI changes were later reverted.