yesBad/pve-storage
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

disk management api: comment how storages are added and cleanup style

Browse Source 
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
This commit is contained in:
Thomas Lamprecht
2023-06-17 14:51:20 +02:00
parent c61e609eb6
commit b89854ad46
4 changed files with 12 additions and 44 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
src/PVE/API2/Disks/Directory.pm
View File

@ -221,13 +221,9 @@ __PACKAGE__->register_method ({
my $verify_params = [qw(path)]; my $verify_params = [qw(path)];
if ($param->{add_storage}) { if ($param->{add_storage}) {
# reserve the name and add as disabled, will be enabled below if creation works out
PVE::API2::Storage::Config->create_or_update( PVE::API2::Storage::Config->create_or_update(
$name, $name, $node, $storage_params, $verify_params, 1);
$node,
$storage_params,
$verify_params,
1,
);
} }
my $mounted = PVE::Diskmanage::mounted_paths(); my $mounted = PVE::Diskmanage::mounted_paths();
@ -306,11 +302,7 @@ __PACKAGE__->register_method ({
if ($param->{add_storage}) { if ($param->{add_storage}) {
PVE::API2::Storage::Config->create_or_update( PVE::API2::Storage::Config->create_or_update(
$name, $name, $node, $storage_params, $verify_params);
$node,
$storage_params,
$verify_params,
);
} }
}); });
}; };

14
src/PVE/API2/Disks/LVM.pm
View File

@ -162,13 +162,9 @@ __PACKAGE__->register_method ({
my $verify_params = [qw(vgname)]; my $verify_params = [qw(vgname)];
if ($param->{add_storage}) { if ($param->{add_storage}) {
# reserve the name and add as disabled, will be enabled below if creation works out
PVE::API2::Storage::Config->create_or_update( PVE::API2::Storage::Config->create_or_update(
$name, $name, $node, $storage_params, $verify_params, 1);
$node,
$storage_params,
$verify_params,
1,
);
} }
my $worker = sub { my $worker = sub {
@ -188,11 +184,7 @@ __PACKAGE__->register_method ({
if ($param->{add_storage}) { if ($param->{add_storage}) {
PVE::API2::Storage::Config->create_or_update( PVE::API2::Storage::Config->create_or_update(
$name, $name, $node, $storage_params, $verify_params);
$node,
$storage_params,
$verify_params,
);
} }
}); });
}; };

14
src/PVE/API2/Disks/LVMThin.pm
View File

@ -120,13 +120,9 @@ __PACKAGE__->register_method ({
my $verify_params = [qw(vgname thinpool)]; my $verify_params = [qw(vgname thinpool)];
if ($param->{add_storage}) { if ($param->{add_storage}) {
# reserve the name and add as disabled, will be enabled below if creation works out
PVE::API2::Storage::Config->create_or_update( PVE::API2::Storage::Config->create_or_update(
$name, $name, $node, $storage_params, $verify_params, 1);
$node,
$storage_params,
$verify_params,
1,
);
} }
my $worker = sub { my $worker = sub {
@ -167,11 +163,7 @@ __PACKAGE__->register_method ({
if ($param->{add_storage}) { if ($param->{add_storage}) {
PVE::API2::Storage::Config->create_or_update( PVE::API2::Storage::Config->create_or_update(
$name, $name, $node, $storage_params, $verify_params);
$node,
$storage_params,
$verify_params,
);
} }
}); });
}; };

14
src/PVE/API2/Disks/ZFS.pm
View File

@ -383,13 +383,9 @@ __PACKAGE__->register_method ({
my $verify_params = [qw(pool)]; my $verify_params = [qw(pool)];
if ($param->{add_storage}) { if ($param->{add_storage}) {
# reserve the name and add as disabled, will be enabled below if creation works out
PVE::API2::Storage::Config->create_or_update( PVE::API2::Storage::Config->create_or_update(
$name, $name, $node, $storage_params, $verify_params, 1);
$node,
$storage_params,
$verify_params,
1,
);
} }
my $pools = get_pool_data(); my $pools = get_pool_data();
@ -495,11 +491,7 @@ __PACKAGE__->register_method ({
if ($param->{add_storage}) { if ($param->{add_storage}) {
PVE::API2::Storage::Config->create_or_update( PVE::API2::Storage::Config->create_or_update(
$name, $name, $node, $storage_params, $verify_params);
$node,
$storage_params,
$verify_params,
);
} }
}; };