str/bytes compatibility for PyNVML device name

[jrbourbeau]

No description provided.

[jrbourbeau]

We've started seeing the following test failures in gpuCI (for example, see this build)

14:55:11 FAILED distributed/diagnostics/tests/test_nvml.py::test_gpu_metrics - AttributeError: 'str' object has no attribute 'decode'
14:55:11 FAILED distributed/diagnostics/tests/test_nvml.py::test_gpu_monitoring_recent - AttributeError: 'str' object has no attribute 'decode'

Related to #7544

cc @pentschev @jacobtomlinson @wence-

[github-actions]

Unit Test Results

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

       24 files  ±0         24 suites  ±0   10h 4m 27s ⏱️ - 2m 20s
  3 337 tests ±0    3 231 ✔️  - 3     103 💤 ±0  3 ❌ +3 
39 343 runs  ±0  37 477 ✔️  - 4  1 863 💤 +1  3 ❌ +3 

For more details on these failures, see this check.

Results for commit 9d0b2f7. ± Comparison against base commit 242079d.

[quasiben]

The error might be due to a recent update in PyNVML. xref: rapidsai/dask-cuda#1118

@wence- @charlesbluca do you have opinions here ?

[wence-]

Thanks @jrbourbeau, and sorry for missing this. This looks right since it now just asks the nvml diagnostics module to decode the name (which handles the str/bytes distinction).

[jrbourbeau]

No worries @wence-! Thanks for reviewing