]> git.proxmox.com Git - pve-storage.git/commitdiff
lvm: wipe signatures on lvcreate
authorStoiko Ivanov <s.ivanov@proxmox.com>
Tue, 6 Jul 2021 09:50:26 +0000 (11:50 +0200)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Tue, 6 Jul 2021 10:39:50 +0000 (12:39 +0200)
With PVE 7.0 we use upstream's lvm2 packages, which seem to detect
'more' signatures (and refuse creating lvs when they are present)

This prevents creating new disks on LVM (thick) storages as reported
on pve-user [0].

Adding -Wy to wipe signatures, and --yes (to actually wipe them
instead of prompting) fixes the aborted lvcreate.

Adding only to LVMPlugin and not to the lvcreate calls in
LvmThinPlugin, since I assume (and my quick tests confirm) that thin
pools are not affected by this issue..

Tested on a virtual test-setup with a LVM storage on a (virtual) iscsi
target and a local lvmthin storage.

[0] https://lists.proxmox.com/pipermail/pve-user/2021-July/172660.html

Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
PVE/Storage/LVMPlugin.pm

index 039bfc13bb04806f3e03242843637513cd6b90bd..139d391eacd3befc6b714f2ed5792f085a68ade3 100644 (file)
@@ -330,7 +330,7 @@ sub lvcreate {
        $size .= "k"; # default to kilobytes
     }
 
-    my $cmd = ['/sbin/lvcreate', '-aly', '--size', $size, '--name', $name];
+    my $cmd = ['/sbin/lvcreate', '-aly', '-Wy', '--yes', '--size', $size, '--name', $name];
     for my $tag (@$tags) {
        push @$cmd, '--addtag', $tag;
     }