api/config update: indentation and whitespace fixes

[pve-storage.git] / PVE / API2 / Storage / Config.pm

diff --git a/PVE/API2/Storage/Config.pm b/PVE/API2/Storage/Config.pm
index bb9d0a548f4c6569086b89656daa068e04ca9659..e00dc95536f7535f1edf48561830bcf82b403836 100755 (executable)
--- a/PVE/API2/Storage/Config.pm
+++ b/PVE/API2/Storage/Config.pm
@@ -39,20 +39,20 @@ my $api_storage_config = sub {
 };
 
 __PACKAGE__->register_method ({
-    name => 'index', 
+    name => 'index',
     path => '',
     method => 'GET',
     description => "Storage index.",
-    permissions => { 
+    permissions => {
        description => "Only list entries where you have 'Datastore.Audit' or 'Datastore.AllocateSpace' permissions on '/storage/<storage>'",
        user => 'all',
     },
     parameters => {
-       additionalProperties => 0,
+       additionalProperties => 0,
        properties => {
-           type => { 
+           type => {
                description => "Only list storage of specific type",
-               type => 'string', 
+               type => 'string',
                enum => $storage_type_enum,
                optional => 1,
            },
@@ -90,15 +90,15 @@ __PACKAGE__->register_method ({
     }});
 
 __PACKAGE__->register_method ({
-    name => 'read', 
+    name => 'read',
     path => '{storage}',
     method => 'GET',
     description => "Read storage configuration.",
-    permissions => { 
+    permissions => {
        check => ['perm', '/storage/{storage}', ['Datastore.Allocate']],
     },
     parameters => {
-       additionalProperties => 0,
+       additionalProperties => 0,
        properties => {
            storage => get_standard_option('pve-storage-id'),
        },
@@ -115,10 +115,10 @@ __PACKAGE__->register_method ({
 __PACKAGE__->register_method ({
     name => 'create',
     protected => 1,
-    path => '', 
+    path => '',
     method => 'POST',
     description => "Create a new storage.",
-    permissions => { 
+    permissions => {
        check => ['perm', '/storage', ['Datastore.Allocate']],
     },
     parameters => PVE::Storage::Plugin->createSchema(),
@@ -143,10 +143,6 @@ __PACKAGE__->register_method ({
            }
        }
 
-       if ($param->{portal}) {
-           $param->{portal} = PVE::Storage::resolv_portal($param->{portal});
-       }
-
        my $plugin = PVE::Storage::Plugin->lookup($type);
        my $opts = $plugin->check_config($storeid, $param, 1, 1);
 
@@ -163,27 +159,6 @@ __PACKAGE__->register_method ({
 
                $plugin->on_add_hook($storeid, $opts, password => $password);
 
-               my $cred_file = undef;
-
-               if ($type eq 'lvm' && $opts->{base}) {
-
-                   my ($baseid, $volname) = PVE::Storage::parse_volume_id($opts->{base});
-
-                   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
-                   if (!($basecfg->{type} eq 'iscsi')) {
-                       die "unsupported base type '$basecfg->{type}'";
-                   }
-
-                   my $path = PVE::Storage::path($cfg, $opts->{base});
-
-                   PVE::Storage::activate_storage($cfg, $baseid);
-
-                   PVE::Storage::LVMPlugin::lvm_create_volume_group($path, $opts->{vgname}, $opts->{shared});
-               }
-
                eval {
                    # try to activate if enabled on local node,
                    # we only do this to detect errors/problems sooner
@@ -194,12 +169,11 @@ __PACKAGE__->register_method ({
                if(my $err = $@) {
                    eval { $plugin->on_delete_hook($storeid, $opts) };
                    warn "$@\n" if $@;
-                   unlink $cred_file if defined($cred_file);
                    die $err;
                }
 
                PVE::Storage::write_config($cfg);
-           
+
            }, "create storage failed");
 
        return undef;
@@ -211,7 +185,7 @@ __PACKAGE__->register_method ({
     path => '{storage}',
     method => 'PUT',
     description => "Update storage configuration.",
-    permissions => { 
+    permissions => {
        check => ['perm', '/storage', ['Datastore.Allocate']],
     },
     parameters => PVE::Storage::Plugin->updateSchema(),
@@ -221,9 +195,9 @@ __PACKAGE__->register_method ({
 
        my $storeid = extract_param($param, 'storage');
        my $digest = extract_param($param, 'digest');
+       my $delete = extract_param($param, 'delete');
 
-        PVE::Storage::lock_storage_config(
-        sub {
+        PVE::Storage::lock_storage_config(sub {
 
            my $cfg = PVE::Storage::config();
 
@@ -234,13 +208,26 @@ __PACKAGE__->register_method ({
            my $plugin = PVE::Storage::Plugin->lookup($scfg->{type});
            my $opts = $plugin->check_config($storeid, $param, 0, 1);
 
-           foreach my $k (%$opts) {
+           if ($delete) {
+               my $options = $plugin->private()->{options}->{$scfg->{type}};
+               foreach my $k (PVE::Tools::split_list($delete)) {
+                   my $d = $options->{$k} || die "no such option '$k'\n";
+                   die "unable to delete required option '$k'\n" if !$d->{optional};
+                   die "unable to delete fixed option '$k'\n" if $d->{fixed};
+                   die "cannot set and delete property '$k' at the same time!\n"
+                       if defined($opts->{$k});
+
+                   delete $scfg->{$k};
+               }
+           }
+
+           for my $k (keys %$opts) {
                $scfg->{$k} = $opts->{$k};
            }
 
            PVE::Storage::write_config($cfg);
 
-           }, "update storage failed");
+       }, "update storage failed");
 
        return undef;
     }});
@@ -251,12 +238,12 @@ __PACKAGE__->register_method ({
     path => '{storage}', # /storage/config/{storage}
     method => 'DELETE',
     description => "Delete storage configuration.",
-    permissions => { 
+    permissions => {
        check => ['perm', '/storage', ['Datastore.Allocate']],
     },
     parameters => {
-       additionalProperties => 0,
-       properties => { 
+       additionalProperties => 0,
+       properties => {
            storage => get_standard_option('pve-storage-id', {
                 completion => \&PVE::Storage::complete_storage,
             }),