[pve-devel] [PATCH storage] lvm: Fix #3159: Show RAID LVs as storage content

Dominic Jäger d.jaeger at proxmox.com
Wed Jan 13 13:19:54 CET 2021


LVM RAID logical volumes (including mirrors) can be valid disk images, so they
should show up in storage content listings (for example pvesm list).

Including LV types is safer than excluding, especially because of possible
additional types in the future.

Co-developed-by: Fabian Grünbichler <f.gruenbichler at proxmox.com>
Signed-off-by: Dominic Jäger <d.jaeger at proxmox.com>
---
 PVE/Storage/LVMPlugin.pm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/PVE/Storage/LVMPlugin.pm b/PVE/Storage/LVMPlugin.pm
index 73e8e48..df49b76 100644
--- a/PVE/Storage/LVMPlugin.pm
+++ b/PVE/Storage/LVMPlugin.pm
@@ -446,7 +446,8 @@ sub list_images {
 
 	    next if $scfg->{tagged_only} && !&$check_tags($info->{tags});
 
-	    next if $info->{lv_type} ne '-';
+	    # Allow mirrored and RAID LVs
+	    next if $info->{lv_type} !~ m/^[-mMrR]$/;
 
 	    my $volid = "$storeid:$volname";
 
-- 
2.20.1




More information about the pve-devel mailing list