Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package lvm2 for openSUSE:Factory checked in 
at 2023-06-27 23:16:12
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/lvm2 (Old)
 and      /work/SRC/openSUSE:Factory/.lvm2.new.15902 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "lvm2"

Tue Jun 27 23:16:12 2023 rev:160 rq:1095569 version:2.03.16

Changes:
--------
--- /work/SRC/openSUSE:Factory/lvm2/lvm2.changes        2023-01-07 
17:16:16.520924959 +0100
+++ /work/SRC/openSUSE:Factory/.lvm2.new.15902/lvm2.changes     2023-06-27 
23:16:15.879249662 +0200
@@ -1,0 +2,6 @@
+Fri Jun 23 06:38:00 UTC 2023 - Heming Zhao <heming.z...@suse.com>
+
+- multipath_component_detection = 0 in lvm.conf does not have any effect 
(bsc#1212613)
+  - bug-1212613_apply-multipath_component_detection-0-to-duplicate-P.patch
+
+-------------------------------------------------------------------

New:
----
  bug-1212613_apply-multipath_component_detection-0-to-duplicate-P.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ lvm2.spec ++++++
--- /var/tmp/diff_new_pack.MBP3pS/_old  2023-06-27 23:16:17.271257855 +0200
+++ /var/tmp/diff_new_pack.MBP3pS/_new  2023-06-27 23:16:17.275257878 +0200
@@ -1,7 +1,7 @@
 #
 # spec file
 #
-# Copyright (c) 2022 SUSE LLC
+# Copyright (c) 2023 SUSE LLC
 #
 # All modifications and additions to the file contributed by third parties
 # remain the property of their copyright owners, unless otherwise agreed
@@ -86,6 +86,7 @@
 Patch0013:      0013-vgimportdevices-change-result-when-devices-are-not-a.patch
 Patch0014:      0014-vgimportdevices-fix-locking-when-creating-devices-fi.patch
 Patch0015:      
bug-1203216_lvmlockd-purge-the-lock-resources-left-in-previous-l.patch
+Patch0016:      
bug-1212613_apply-multipath_component_detection-0-to-duplicate-P.patch
 # SUSE patches: 1000+ for LVM
 # Never upstream
 Patch1001:      cmirrord_remove_date_time_from_compilation.patch
@@ -107,6 +108,7 @@
 BuildRequires:  libaio-devel
 BuildRequires:  pkgconfig
 BuildRequires:  thin-provisioning-tools >= %{thin_provisioning_version}
+BuildRequires:  pkgconfig(libsystemd)
 BuildRequires:  pkgconfig(libudev)
 Requires:       device-mapper >= %{device_mapper_version}
 Requires:       modutils
@@ -121,13 +123,11 @@
 BuildRequires:  suse-module-tools
 BuildRequires:  pkgconfig(libselinux)
 BuildRequires:  pkgconfig(libsepol)
-BuildRequires:  pkgconfig(systemd)
 %else
 BuildRequires:  libcorosync-devel
 BuildRequires:  pkgconfig(blkid)
 %if %{with lockd}
 BuildRequires:  libdlm-devel >= %{dlm_version}
-BuildRequires:  pkgconfig(libsystemd)
 %if 0%{_supportsanlock} == 1
 BuildRequires:  sanlock-devel >= %{sanlock_version}
 %endif
@@ -135,7 +135,6 @@
 BuildRequires:  gcc-c++
 BuildRequires:  libselinux-devel
 BuildRequires:  readline-devel
-BuildRequires:  pkgconfig(systemd)
 BuildRequires:  pkgconfig(udev)
 %endif
 %endif
@@ -161,6 +160,7 @@
 %patch0013 -p1
 %patch0014 -p1
 %patch0015 -p1
+%patch0016 -p1
 %patch1001 -p1
 %patch1002 -p1
 %patch1003 -p1


++++++ bug-1212613_apply-multipath_component_detection-0-to-duplicate-P.patch 
++++++
>From 99ce09ae778c2cc4aa2611e425bba5287b8b9513 Mon Sep 17 00:00:00 2001
From: David Teigland <teigl...@redhat.com>
Date: Mon, 25 Jul 2022 13:50:43 -0500
Subject: [PATCH] apply multipath_component_detection=0 to duplicate PV
 handling

multipath_component_detection=0 has always applied to the filter-based
component detection.  Also apply this setting to the duplicate-PV
handling which also eliminates multipath components (based on duplicate
PVs having the same wwid.)
---
 lib/cache/lvmcache.c                  |  3 +++
 test/shell/duplicate-pvs-multipath.sh | 10 +++++++---
 2 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/lib/cache/lvmcache.c b/lib/cache/lvmcache.c
index a1c4a61c82fa..00916885c509 100644
--- a/lib/cache/lvmcache.c
+++ b/lib/cache/lvmcache.c
@@ -652,6 +652,9 @@ static int _all_multipath_components(struct cmd_context 
*cmd, struct lvmcache_in
 
        *dev_mpath = NULL;
 
+       if (!find_config_tree_bool(cmd, 
devices_multipath_component_detection_CFG, NULL))
+               return 0;
+
        /* This function only makes sense with more than one dev. */
        if ((info && dm_list_empty(altdevs)) || (!info && 
(dm_list_size(altdevs) == 1))) {
                log_debug("Skip multipath component checks with single device 
for PVID %s", pvid);
diff --git a/test/shell/duplicate-pvs-multipath.sh 
b/test/shell/duplicate-pvs-multipath.sh
index 59c15b0d4f9b..bc98d2d5aa8c 100644
--- a/test/shell/duplicate-pvs-multipath.sh
+++ b/test/shell/duplicate-pvs-multipath.sh
@@ -24,9 +24,13 @@ modprobe --dry-run scsi_debug || skip
 multipath -l || skip
 multipath -l | grep scsi_debug && skip
 
-# Turn off multipath_component_detection so that the duplicate
-# resolution of mpath components is used.
-aux lvmconf 'devices/multipath_component_detection = 0'
+# FIXME: setting multipath_component_detection=0 now also disables
+# the wwid-based mpath component detection, so this test will need
+# to find another way to disable only the filter-mpath code (using
+# sysfs and multipath/wwids) while keeping the code enabled that
+# eliminates duplicates based on their matching wwids which this
+# tries to test.
+
 # Prevent wwids from being used for filtering.
 aux lvmconf 'devices/multipath_wwids_file = "/dev/null"'
 # Need to use /dev/mapper/mpath
-- 
2.35.3


++++++ lvm.conf ++++++
--- /var/tmp/diff_new_pack.MBP3pS/_old  2023-06-27 23:16:17.435258820 +0200
+++ /var/tmp/diff_new_pack.MBP3pS/_new  2023-06-27 23:16:17.435258820 +0200
@@ -2421,7 +2421,7 @@
        # Configuration option dmeventd/executable.
        # The full path to the dmeventd binary.
        # This configuration option has an automatic default value.
-       # executable = ""
+       # executable = "/usr/sbin/dmeventd"
 }
 
 # Configuration section tags.

Reply via email to