lvm: Require higher version of LVM for lvm.conf parsing

The --valuesonly option we use is not available on LVM < 2.03.17

Author: vojtechtrefny

src/plugins/lvm-dbus.c

@@ -300,7 +300,7 @@ static GMutex deps_check_lock;
 static const UtilDep deps[DEPS_LAST] = {
     {"lvm", LVM_MIN_VERSION, "version", "LVM version:\\s+([\\d\\.]+)"},
     {"lvmdevices", NULL, NULL, NULL},
-    {"lvmconfig", NULL, NULL, NULL},
+    {"lvmconfig", "2.03.17", "version", "LVM version:\\s+([\\d\\.]+)"},
 };
 
 #define DBUS_DEPS_LVMDBUSD 0

src/plugins/lvm.c

@@ -307,11 +307,14 @@ static GMutex deps_check_lock;
 #define DEPS_LVM_MASK (1 << DEPS_LVM)
 #define DEPS_LVMDEVICES 1
 #define DEPS_LVMDEVICES_MASK (1 << DEPS_LVMDEVICES)
-#define DEPS_LAST 2
+#define DEPS_LVMCONFIG 2
+#define DEPS_LVMCONFIG_MASK (1 << DEPS_LVMCONFIG)
+#define DEPS_LAST 3
 
 static const UtilDep deps[DEPS_LAST] = {
     {"lvm", LVM_MIN_VERSION, "version", "LVM version:\\s+([\\d\\.]+)"},
     {"lvmdevices", NULL, NULL, NULL},
+    {"lvmconfig", "2.03.17", "version", "LVM version:\\s+([\\d\\.]+)"},
 };
 
 #define FEATURES_VDO 0

tests/skip.yml

@@ -59,3 +59,9 @@
 - test: lvm_dbus_tests.LvmTestPartialLVs
   skip_on:
     - reason: "LVM DBus doesn't support LV repair yet"
+
+- test: (lvm_test|lvm_dbus_tests).LvmNoDevTestCase.test_lvm_config
+  skip_on:
+    - distro: "debian"
+      version: "12"
+      reason: "LVM >= 2.03.17 needed for LVM config parsing with --valuesonly"