disks: allow add_storage for already configured local storage

[pve-storage.git] / PVE / API2 / Disks / Directory.pm

diff --git a/PVE/API2/Disks/Directory.pm b/PVE/API2/Disks/Directory.pm
index e2272fbb301837b09e393fec558e4f42c2b6f134..4fdb06874d0754a55ac4e9beba80664db45960b9 100644 (file)
--- a/PVE/API2/Disks/Directory.pm
+++ b/PVE/API2/Disks/Directory.pm
@@ -209,7 +209,26 @@ __PACKAGE__->register_method ({
 
        $dev = PVE::Diskmanage::verify_blockdev_path($dev);
        PVE::Diskmanage::assert_disk_unused($dev);
-       PVE::Storage::assert_sid_unused($name) if $param->{add_storage};
+
+       my $storage_params = {
+           type => 'dir',
+           storage => $name,
+           content => 'rootdir,images,iso,backup,vztmpl,snippets',
+           is_mountpoint => 1,
+           path => $path,
+           nodes => $node,
+       };
+       my $verify_params = [qw(path)];
+
+       if ($param->{add_storage}) {
+           PVE::API2::Storage::Config->create_or_update(
+               $name,
+               $node,
+               $storage_params,
+               $verify_params,
+               1,
+           );
+       }
 
        my $mounted = PVE::Diskmanage::mounted_paths();
        die "the path for '${name}' is already mounted: ${path} ($mounted->{$path})\n"
@@ -286,16 +305,12 @@ __PACKAGE__->register_method ({
                run_command(['systemctl', 'start', $mountunitname]);
 
                if ($param->{add_storage}) {
-                   my $storage_params = {
-                       type => 'dir',
-                       storage => $name,
-                       content => 'rootdir,images,iso,backup,vztmpl,snippets',
-                       is_mountpoint => 1,
-                       path => $path,
-                       nodes => $node,
-                   };
-
-                   PVE::API2::Storage::Config->create($storage_params);
+                   PVE::API2::Storage::Config->create_or_update(
+                       $name,
+                       $node,
+                       $storage_params,
+                       $verify_params,
+                   );
                }
            });
        };