Hello community,

here is the log from the commit of package lvm2 for openSUSE:Factory checked in 
at 2020-06-03 20:28:42
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/lvm2 (Old)
 and      /work/SRC/openSUSE:Factory/.lvm2.new.3606 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "lvm2"

Wed Jun  3 20:28:42 2020 rev:136 rq:809718 version:2.03.05

Changes:
--------
--- /work/SRC/openSUSE:Factory/lvm2/lvm2.changes        2020-05-14 
23:22:35.452692446 +0200
+++ /work/SRC/openSUSE:Factory/.lvm2.new.3606/lvm2.changes      2020-06-03 
20:28:51.896506477 +0200
@@ -1,0 +2,6 @@
+Wed May 27 13:00:58 UTC 2020 - heming.z...@suse.com
+
+- removing LVM cache with cache volume does not remove the cache volume 
(bsc#1171907)
+  + bug-1171907-lvremove-remove-attached-cachevol-with-removed-LV.patch
+
+-------------------------------------------------------------------

New:
----
  bug-1171907-lvremove-remove-attached-cachevol-with-removed-LV.patch

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

Other differences:
------------------
++++++ lvm2.spec ++++++
--- /var/tmp/diff_new_pack.6g2Akf/_old  2020-06-03 20:28:53.292510721 +0200
+++ /var/tmp/diff_new_pack.6g2Akf/_new  2020-06-03 20:28:53.292510721 +0200
@@ -93,6 +93,7 @@
 Patch0036:      
bug-1150021_05-bcache-bcache_invalidate_fd-only-remove-prefixes-on.patch
 Patch0037:      bug-1150021_06-fix-dev_unset_last_byte-after-write-error.patch
 Patch0038:      bug-1157736-add-suggestion-message-for-mirror-LVs.patch
+Patch0039:      
bug-1171907-lvremove-remove-attached-cachevol-with-removed-LV.patch
 # SUSE patches: 1000+ for LVM
 # Never upstream
 Patch1001:      cmirrord_remove_date_time_from_compilation.patch
@@ -185,6 +186,7 @@
 %patch0036 -p1
 %patch0037 -p1
 %patch0038 -p1
+%patch0039 -p1
 %patch1001 -p1
 %patch1002 -p1
 %patch1003 -p1


++++++ bug-1171907-lvremove-remove-attached-cachevol-with-removed-LV.patch 
++++++
change this patch
from: if (!lv_detach_cache_vol(lv, 0)) {
to  : if (!lv_detach_cache_vol(lv)) {
by heming.z...@suse.com

---
>From 56aadd7fe2a1d24043ea9d06543c29317ac1cc58 Mon Sep 17 00:00:00 2001
From: David Teigland <teigl...@redhat.com>
Date: Tue, 24 Sep 2019 13:46:40 -0500
Subject: [PATCH] lvremove: remove attached cachevol with removed LV

When an LV is removed that has an attached cachevol,
also remove the cachevol LV.
---
 lib/metadata/lv_manip.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/lib/metadata/lv_manip.c b/lib/metadata/lv_manip.c
index 79e3d15..f404555 100644
--- a/lib/metadata/lv_manip.c
+++ b/lib/metadata/lv_manip.c
@@ -6283,10 +6283,16 @@ int lv_remove_single(struct cmd_context *cmd, struct 
logical_volume *lv,
                return_0;
 
        if (lv_is_cache(lv) && lv_is_cache_vol(first_seg(lv)->pool_lv)) {
+               struct logical_volume *cachevol_lv = first_seg(lv)->pool_lv;
+
                if (!lv_detach_cache_vol(lv)) {
                        log_error("Failed to detach cache from %s", 
display_lvname(lv));
                        return 0;
                }
+               if (!lv_remove_single(cmd, cachevol_lv, force, 
suppress_remove_message)) {
+                       log_error("Failed to remove cachevol %s.", 
display_lvname(cachevol_lv));
+                       return 0;
+               }
        }
 
        /* FIXME Ensure not referred to by another existing LVs */
-- 
1.8.3.1


Reply via email to