From 618bdfd92705da77332d0b4b732f4eab3062093d Mon Sep 17 00:00:00 2001 From: Philipp Hufnagl Date: Tue, 25 Jul 2023 16:37:19 +0200 Subject: [PATCH] clean: fix whitspaces and minor code issues MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit removed Data::Dumper and a newline Signed-off-by: Philipp Hufnagl re-added the newline, but fixed up the indentation at that line.. Signed-off-by: Fabian Grünbichler --- src/PVE/API2/Storage/Status.pm | 18 +++++------ src/PVE/Storage.pm | 55 +++++++++++++++++----------------- 2 files changed, 36 insertions(+), 37 deletions(-) diff --git a/src/PVE/API2/Storage/Status.pm b/src/PVE/API2/Storage/Status.pm index e4ce698..2aaeff6 100644 --- a/src/PVE/API2/Storage/Status.pm +++ b/src/PVE/API2/Storage/Status.pm @@ -53,7 +53,7 @@ __PACKAGE__->register_method ({ protected => 1, proxyto => 'node', parameters => { - additionalProperties => 0, + additionalProperties => 0, properties => { node => get_standard_option('pve-node'), storage => get_standard_option('pve-storage-id', { @@ -204,7 +204,7 @@ __PACKAGE__->register_method ({ check => ['perm', '/storage/{storage}', ['Datastore.Audit', 'Datastore.AllocateSpace'], any => 1], }, parameters => { - additionalProperties => 0, + additionalProperties => 0, properties => { node => get_standard_option('pve-node'), storage => get_standard_option('pve-storage-id'), @@ -248,7 +248,7 @@ __PACKAGE__->register_method ({ protected => 1, proxyto => 'node', parameters => { - additionalProperties => 0, + additionalProperties => 0, properties => { node => get_standard_option('pve-node'), storage => get_standard_option('pve-storage-id'), @@ -284,7 +284,7 @@ __PACKAGE__->register_method ({ protected => 1, proxyto => 'node', parameters => { - additionalProperties => 0, + additionalProperties => 0, properties => { node => get_standard_option('pve-node'), storage => get_standard_option('pve-storage-id'), @@ -295,11 +295,11 @@ __PACKAGE__->register_method ({ }, ds => { description => "The list of datasources you want to display.", - type => 'string', format => 'pve-configid-list', + type => 'string', format => 'pve-configid-list', }, cf => { description => "The RRD consolidation function", - type => 'string', + type => 'string', enum => [ 'AVERAGE', 'MAX' ], optional => 1, }, @@ -330,7 +330,7 @@ __PACKAGE__->register_method ({ protected => 1, proxyto => 'node', parameters => { - additionalProperties => 0, + additionalProperties => 0, properties => { node => get_standard_option('pve-node'), storage => get_standard_option('pve-storage-id'), @@ -341,7 +341,7 @@ __PACKAGE__->register_method ({ }, cf => { description => "The RRD consolidation function", - type => 'string', + type => 'string', enum => [ 'AVERAGE', 'MAX' ], optional => 1, }, @@ -374,7 +374,7 @@ __PACKAGE__->register_method ({ }, protected => 1, parameters => { - additionalProperties => 0, + additionalProperties => 0, properties => { node => get_standard_option('pve-node'), storage => get_standard_option('pve-storage-id'), diff --git a/src/PVE/Storage.pm b/src/PVE/Storage.pm index b99ed35..a4d85e1 100755 --- a/src/PVE/Storage.pm +++ b/src/PVE/Storage.pm @@ -2,7 +2,6 @@ package PVE::Storage; use strict; use warnings; -use Data::Dumper; use POSIX; use IO::Select; @@ -304,11 +303,11 @@ sub volume_resize { my ($storeid, $volname) = parse_volume_id($volid, 1); if ($storeid) { - my $scfg = storage_config($cfg, $storeid); - my $plugin = PVE::Storage::Plugin->lookup($scfg->{type}); - return $plugin->volume_resize($scfg, $storeid, $volname, $size, $running); + my $scfg = storage_config($cfg, $storeid); + my $plugin = PVE::Storage::Plugin->lookup($scfg->{type}); + return $plugin->volume_resize($scfg, $storeid, $volname, $size, $running); } elsif ($volid =~ m|^(/.+)$| && -e $volid) { - die "resize file/device '$volid' is not possible\n"; + die "resize file/device '$volid' is not possible\n"; } else { die "unable to parse volume ID '$volid'\n"; } @@ -319,11 +318,11 @@ sub volume_rollback_is_possible { my ($storeid, $volname) = parse_volume_id($volid, 1); if ($storeid) { - my $scfg = storage_config($cfg, $storeid); - my $plugin = PVE::Storage::Plugin->lookup($scfg->{type}); - return $plugin->volume_rollback_is_possible($scfg, $storeid, $volname, $snap, $blockers); + my $scfg = storage_config($cfg, $storeid); + my $plugin = PVE::Storage::Plugin->lookup($scfg->{type}); + return $plugin->volume_rollback_is_possible($scfg, $storeid, $volname, $snap, $blockers); } elsif ($volid =~ m|^(/.+)$| && -e $volid) { - die "snapshot rollback file/device '$volid' is not possible\n"; + die "snapshot rollback file/device '$volid' is not possible\n"; } else { die "unable to parse volume ID '$volid'\n"; } @@ -334,11 +333,11 @@ sub volume_snapshot { my ($storeid, $volname) = parse_volume_id($volid, 1); if ($storeid) { - my $scfg = storage_config($cfg, $storeid); - my $plugin = PVE::Storage::Plugin->lookup($scfg->{type}); - return $plugin->volume_snapshot($scfg, $storeid, $volname, $snap); + my $scfg = storage_config($cfg, $storeid); + my $plugin = PVE::Storage::Plugin->lookup($scfg->{type}); + return $plugin->volume_snapshot($scfg, $storeid, $volname, $snap); } elsif ($volid =~ m|^(/.+)$| && -e $volid) { - die "snapshot file/device '$volid' is not possible\n"; + die "snapshot file/device '$volid' is not possible\n"; } else { die "unable to parse volume ID '$volid'\n"; } @@ -349,12 +348,12 @@ sub volume_snapshot_rollback { my ($storeid, $volname) = parse_volume_id($volid, 1); if ($storeid) { - my $scfg = storage_config($cfg, $storeid); - my $plugin = PVE::Storage::Plugin->lookup($scfg->{type}); + my $scfg = storage_config($cfg, $storeid); + my $plugin = PVE::Storage::Plugin->lookup($scfg->{type}); $plugin->volume_rollback_is_possible($scfg, $storeid, $volname, $snap); - return $plugin->volume_snapshot_rollback($scfg, $storeid, $volname, $snap); + return $plugin->volume_snapshot_rollback($scfg, $storeid, $volname, $snap); } elsif ($volid =~ m|^(/.+)$| && -e $volid) { - die "snapshot rollback file/device '$volid' is not possible\n"; + die "snapshot rollback file/device '$volid' is not possible\n"; } else { die "unable to parse volume ID '$volid'\n"; } @@ -366,11 +365,11 @@ sub volume_snapshot_delete { my ($storeid, $volname) = parse_volume_id($volid, 1); if ($storeid) { - my $scfg = storage_config($cfg, $storeid); - my $plugin = PVE::Storage::Plugin->lookup($scfg->{type}); - return $plugin->volume_snapshot_delete($scfg, $storeid, $volname, $snap, $running); + my $scfg = storage_config($cfg, $storeid); + my $plugin = PVE::Storage::Plugin->lookup($scfg->{type}); + return $plugin->volume_snapshot_delete($scfg, $storeid, $volname, $snap, $running); } elsif ($volid =~ m|^(/.+)$| && -e $volid) { - die "snapshot delete file/device '$volid' is not possible\n"; + die "snapshot delete file/device '$volid' is not possible\n"; } else { die "unable to parse volume ID '$volid'\n"; } @@ -409,9 +408,9 @@ sub volume_has_feature { my ($storeid, $volname) = parse_volume_id($volid, 1); if ($storeid) { - my $scfg = storage_config($cfg, $storeid); - my $plugin = PVE::Storage::Plugin->lookup($scfg->{type}); - return $plugin->volume_has_feature($scfg, $feature, $storeid, $volname, $snap, $running, $opts); + my $scfg = storage_config($cfg, $storeid); + my $plugin = PVE::Storage::Plugin->lookup($scfg->{type}); + return $plugin->volume_has_feature($scfg, $feature, $storeid, $volname, $snap, $running, $opts); } elsif ($volid =~ m|^(/.+)$| && -e $volid) { return undef; } else { @@ -1503,7 +1502,7 @@ sub foreach_volid { foreach my $sid (keys %$list) { foreach my $info (@{$list->{$sid}}) { - my $volid = $info->{volid}; + my $volid = $info->{volid}; my ($sid1, $volname) = parse_volume_id($volid, 1); if ($sid1 && $sid1 eq $sid) { &$func ($volid, $sid, $info); @@ -1827,7 +1826,7 @@ sub volume_export : prototype($$$$$$$) { my $scfg = storage_config($cfg, $storeid); my $plugin = PVE::Storage::Plugin->lookup($scfg->{type}); return $plugin->volume_export($scfg, $storeid, $fh, $volname, $format, - $snapshot, $base_snapshot, $with_snapshots); + $snapshot, $base_snapshot, $with_snapshots); } sub volume_import : prototype($$$$$$$$) { @@ -1858,8 +1857,8 @@ sub volume_export_formats : prototype($$$$$) { my $scfg = storage_config($cfg, $storeid); my $plugin = PVE::Storage::Plugin->lookup($scfg->{type}); return $plugin->volume_export_formats($scfg, $storeid, $volname, - $snapshot, $base_snapshot, - $with_snapshots); + $snapshot, $base_snapshot, + $with_snapshots); } sub volume_import_formats : prototype($$$$$) {