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