[PATCH storage 05/13] lvm: allow a multipath storage as the base device

[Thomas Lamprecht <t.lamprecht@proxmox.com>]

on_add_hook only accepted an iscsi base when creating the volume group.
Accept the new multipath type as well, so a shared VG can sit on a
multipath LUN and inherit its path redundancy.

The map device is assembled on demand by multipathd, so activate the
base volume to ensure the map is present before pvcreate runs on it.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
---
 src/PVE/Storage/LVMPlugin.pm | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/PVE/Storage/LVMPlugin.pm b/src/PVE/Storage/LVMPlugin.pm
index a313ecc..433c942 100644
--- a/src/PVE/Storage/LVMPlugin.pm
+++ b/src/PVE/Storage/LVMPlugin.pm
@@ -485,13 +485,14 @@ sub on_add_hook {
         my $basecfg = PVE::Storage::storage_config($cfg, $baseid, 1);
         die "base storage ID '$baseid' does not exist\n" if !$basecfg;
 
-        # we only support iscsi for now
-        die "unsupported base type '$basecfg->{type}'"
-            if $basecfg->{type} ne 'iscsi';
+        die "unsupported base type '$basecfg->{type}'\n"
+            if $basecfg->{type} ne 'iscsi' && $basecfg->{type} ne 'multipath';
 
         my $path = PVE::Storage::path($cfg, $base);
 
         PVE::Storage::activate_storage($cfg, $baseid);
+        # the multipath map must be assembled before we can pvcreate on it
+        PVE::Storage::activate_volumes($cfg, [$base]) if $basecfg->{type} eq 'multipath';
 
         lvm_create_volume_group($path, $scfg->{vgname}, $scfg->{shared});
     }
-- 
2.47.3