[PATCH v2 2/2] locking: use & install test_libvirt_sanlock.sug for both QEMU & LibXL

Daniel P. Berrangé via Devel posted 2 patches 3 weeks ago
[PATCH v2 2/2] locking: use & install test_libvirt_sanlock.sug for both QEMU & LibXL
Posted by Daniel P. Berrangé via Devel 3 weeks ago
From: Daniel P. Berrangé <berrange@redhat.com>

The RPM specfile was referencing test_libvirt_sanlock.aug in the common
file list, for both QEMU and LibXL. This makes sense since the
sanlock.conf file is cloned for both drivers. The libvirt_sanlock.aug
file, however, was missing a reference to the LibXL copy of the config.

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
 src/locking/libvirt_sanlock.aug |  1 +
 src/locking/meson.build         | 16 ++++++++--------
 2 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/src/locking/libvirt_sanlock.aug b/src/locking/libvirt_sanlock.aug
index 184ea1f181..62d0672952 100644
--- a/src/locking/libvirt_sanlock.aug
+++ b/src/locking/libvirt_sanlock.aug
@@ -33,6 +33,7 @@ module Libvirt_sanlock =
    let lns = ( record | comment | empty ) *
 
    let filter = incl "/etc/libvirt/qemu-sanlock.conf"
+              . incl "/etc/libvirt/libxl-sanlock.conf"
               . Util.stdexcl
 
    let xfm = transform lns filter
diff --git a/src/locking/meson.build b/src/locking/meson.build
index 934e47593b..f8d12c481f 100644
--- a/src/locking/meson.build
+++ b/src/locking/meson.build
@@ -207,6 +207,14 @@ if conf.has('WITH_LIBVIRTD')
 
   if conf.has('WITH_SANLOCK')
     virt_aug_files += files('libvirt_sanlock.aug')
+    virt_test_aug_files += {
+      'name': 'test_libvirt_sanlock.aug',
+      'aug': files('test_libvirt_sanlock.aug.in'),
+      'conf': files('sanlock.conf'),
+      'test_name': 'libvirt_sanlock',
+      'test_srcdir': meson.current_source_dir(),
+      'test_builddir': meson.current_build_dir(),
+    }
 
     if conf.has('WITH_QEMU')
       qemu_sanlock_conf = configure_file(
@@ -215,14 +223,6 @@ if conf.has('WITH_LIBVIRTD')
         configuration: configmake_conf,
       )
       virt_conf_files += qemu_sanlock_conf
-      virt_test_aug_files += {
-        'name': 'test_libvirt_sanlock.aug',
-        'aug': files('test_libvirt_sanlock.aug.in'),
-        'conf': qemu_sanlock_conf,
-        'test_name': 'libvirt_sanlock',
-        'test_srcdir': meson.current_source_dir(),
-        'test_builddir': meson.current_build_dir(),
-      }
     endif
 
     if conf.has('WITH_LIBXL')
-- 
2.50.1