]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
test: skip TEST-84-STORAGETM if running with bugged libnvme
authorLuca Boccassi <luca.boccassi@gmail.com>
Thu, 14 Nov 2024 16:19:25 +0000 (16:19 +0000)
committerLuca Boccassi <luca.boccassi@gmail.com>
Thu, 14 Nov 2024 18:00:35 +0000 (18:00 +0000)
libnvme 1.11 appears to require a kernel built with NVME TLS
kconfigs, and fails hard if it is not, as the expected
privileged keyring '.nvme' is not present. We cannot just
create it from userspace, as privileged keyrings can only
be created by the kernel itself (those starting with '.').

Skip the test if the library exactly matches this version.

https://github.com/linux-nvme/nvme-cli/issues/2573

Fixes https://github.com/systemd/systemd/issues/35130

test/units/TEST-84-STORAGETM.sh

index 1dd2ea61849e69d56f65305f6e9c6ed2838f668a..0020fad92aa35056de70ee32828b83b23692e7f7 100755 (executable)
@@ -3,6 +3,14 @@
 set -eux
 set -o pipefail
 
+if systemd-analyze compare-versions "$(nvme --version | grep libnvme | awk '{print $3}')" eq 1.11; then
+    if grep -q "CONFIG_NVME_TCP_TLS is not set" "/boot/config-$(uname -r)" 2>/dev/null || grep -q "CONFIG_NVME_TCP_TLS is not set" "/usr/lib/modules/$(uname -r)/config" 2>/dev/null; then
+        # See: https://github.com/linux-nvme/nvme-cli/issues/2573
+        echo "nvme-cli is broken and requires TLS support in the kernel" >/skipped
+        exit 77
+    fi
+fi
+
 /usr/lib/systemd/systemd-storagetm --list-devices
 
 modprobe -v nvmet-tcp