[pve-devel] [PATCH storage 2/2] fix #3633: pvesm: use print_api_results

Lorenz Stechauner l.stechauner at proxmox.com
Mon Sep 27 09:09:17 CEST 2021


Signed-off-by: Lorenz Stechauner <l.stechauner at proxmox.com>
---
 PVE/CLI/pvesm.pm | 144 ++++++-----------------------------------------
 1 file changed, 18 insertions(+), 126 deletions(-)

diff --git a/PVE/CLI/pvesm.pm b/PVE/CLI/pvesm.pm
index 7afbe22..7bb363c 100755
--- a/PVE/CLI/pvesm.pm
+++ b/PVE/CLI/pvesm.pm
@@ -178,68 +178,6 @@ __PACKAGE__->register_method ({
 	return;
     }});
 
-my $print_content = sub {
-    my ($list) = @_;
-
-    my ($maxlenname, $maxsize) = (0, 0);
-    foreach my $info (@$list) {
-	my $volid = $info->{volid};
-	my $sidlen =  length ($volid);
-	$maxlenname = $sidlen if $sidlen > $maxlenname;
-	$maxsize = $info->{size} if ($info->{size} // 0) > $maxsize;
-    }
-    my $sizemaxdigits = length($maxsize);
-
-    my $basefmt = "%-${maxlenname}s %-7s %-9s %${sizemaxdigits}s";
-    printf "$basefmt %s\n", "Volid", "Format", "Type", "Size", "VMID";
-
-    foreach my $info (@$list) {
-	next if !$info->{vmid};
-	my $volid = $info->{volid};
-
-	printf "$basefmt %d\n", $volid, $info->{format}, $info->{content}, $info->{size}, $info->{vmid};
-    }
-
-    foreach my $info (sort { $a->{format} cmp $b->{format} } @$list) {
-	next if $info->{vmid};
-	my $volid = $info->{volid};
-
-	printf "$basefmt\n", $volid, $info->{format}, $info->{content}, $info->{size};
-    }
-};
-
-my $print_status = sub {
-    my $res = shift;
-
-    my $maxlen = 0;
-    foreach my $res (@$res) {
-	my $storeid = $res->{storage};
-	$maxlen = length ($storeid) if length ($storeid) > $maxlen;
-    }
-    $maxlen+=1;
-
-    printf "%-${maxlen}s %10s %10s %15s %15s %15s %8s\n", 'Name', 'Type',
-	'Status', 'Total', 'Used', 'Available', '%';
-
-    foreach my $res (sort { $a->{storage} cmp $b->{storage} } @$res) {
-	my $storeid = $res->{storage};
-
-	my $active = $res->{active} ? 'active' : 'inactive';
-	my ($per, $per_fmt) = (0, '% 7.2f%%');
-	$per = ($res->{used}*100)/$res->{total} if $res->{total} > 0;
-
-	if (!$res->{enabled}) {
-	    $per = 'N/A';
-	    $per_fmt = '% 8s';
-	    $active = 'disabled';
-	}
-
-	printf "%-${maxlen}s %10s %10s %15d %15d %15d $per_fmt\n", $storeid,
-	    $res->{type}, $active, $res->{total}/1024, $res->{used}/1024,
-	    $res->{avail}/1024, $per;
-    }
-};
-
 __PACKAGE__->register_method ({
     name => 'export',
     path => 'export',
@@ -594,14 +532,16 @@ our $cmddef = {
 	'index',
 	[],
 	{ node => $nodename },
-	$print_status,
+	$print_api_result,
+	$PVE::RESTHandler::standard_output_options,
     ],
     list => [
 	"PVE::API2::Storage::Content",
 	'index',
 	['storage'],
 	{ node => $nodename },
-	$print_content,
+	$print_api_result,
+	$PVE::RESTHandler::standard_output_options,
     ],
     alloc => [
 	"PVE::API2::Storage::Content",
@@ -625,91 +565,48 @@ our $cmddef = {
 	    'nfsscan',
 	    ['server'],
 	    { node => $nodename },
-	    sub  {
-		my $res = shift;
-
-		my $maxlen = 0;
-		foreach my $rec (@$res) {
-		    my $len = length ($rec->{path});
-		    $maxlen = $len if $len > $maxlen;
-		}
-		foreach my $rec (@$res) {
-		    printf "%-${maxlen}s %s\n", $rec->{path}, $rec->{options};
-		}
-	    },
+	    $print_api_result,
+	    $PVE::RESTHandler::standard_output_options,
 	],
 	cifs => [
 	    "PVE::API2::Storage::Scan",
 	    'cifsscan',
 	    ['server'],
 	    { node => $nodename },
-	    sub  {
-		my $res = shift;
-
-		my $maxlen = 0;
-		foreach my $rec (@$res) {
-		    my $len = length ($rec->{share});
-		    $maxlen = $len if $len > $maxlen;
-		}
-		foreach my $rec (@$res) {
-		    printf "%-${maxlen}s %s\n", $rec->{share}, $rec->{description};
-		}
-	    },
+	    $print_api_result,
+	    $PVE::RESTHandler::standard_output_options,
 	],
 	glusterfs => [
 	    "PVE::API2::Storage::Scan",
 	    'glusterfsscan',
 	    ['server'],
 	    { node => $nodename },
-	    sub  {
-		my $res = shift;
-
-		foreach my $rec (@$res) {
-		    printf "%s\n", $rec->{volname};
-		}
-	    },
+	    $print_api_result,
+	    $PVE::RESTHandler::standard_output_options,
 	],
 	iscsi => [
 	    "PVE::API2::Storage::Scan",
 	    'iscsiscan',
 	    ['portal'],
 	    { node => $nodename },
-	    sub  {
-		my $res = shift;
-
-		my $maxlen = 0;
-		foreach my $rec (@$res) {
-		    my $len = length ($rec->{target});
-		    $maxlen = $len if $len > $maxlen;
-		}
-		foreach my $rec (@$res) {
-		    printf "%-${maxlen}s %s\n", $rec->{target}, $rec->{portal};
-		}
-	    },
+	    $print_api_result,
+	    $PVE::RESTHandler::standard_output_options,
 	],
 	lvm => [
 	    "PVE::API2::Storage::Scan",
 	    'lvmscan',
 	    [],
 	    { node => $nodename },
-	    sub  {
-		my $res = shift;
-		foreach my $rec (@$res) {
-		    printf "$rec->{vg}\n";
-		}
-	    },
+	    $print_api_result,
+	    $PVE::RESTHandler::standard_output_options,
 	],
 	lvmthin => [
 	    "PVE::API2::Storage::Scan",
 	    'lvmthinscan',
 	    ['vg'],
 	    { node => $nodename },
-	    sub  {
-		my $res = shift;
-		foreach my $rec (@$res) {
-		    printf "$rec->{lv}\n";
-		}
-	    },
+	    $print_api_result,
+	    $PVE::RESTHandler::standard_output_options,
 	],
 	pbs => [
 	    "PVE::API2::Storage::Scan",
@@ -724,13 +621,8 @@ our $cmddef = {
 	    'zfsscan',
 	    [],
 	    { node => $nodename },
-	    sub  {
-		my $res = shift;
-
-		foreach my $rec (@$res) {
-		    printf "$rec->{pool}\n";
-		}
-	    },
+	    $print_api_result,
+	    $PVE::RESTHandler::standard_output_options,
 	],
     },
     nfsscan => { alias => 'scan nfs' },
-- 
2.30.2





More information about the pve-devel mailing list