[pve-devel] [PATCH] vzdump : fix check if volid exist for "virtual" storage : rbd, sheepdog, nexenta
Dietmar Maurer
dietmar at proxmox.com
Mon Feb 18 06:40:32 CET 2013
> - die "no such volume '$volid'\n" if ! -e $path;
> + if (! -e $path){
> + my $dl = PVE::Storage::vdisk_list($self->{storecfg}, $storeid, undef);
> + my $foundvolid = undef;
> +
> + PVE::Storage::foreach_volid($dl, sub {
> + my ($volumeid) = @_;
> + if($volumeid eq $volid) {
> + $foundvolid = 1;
> + return;
> + }
> + });
> + die "no such volume '$volid'\n" if !$foundvolid;
> + }
>
> my ($size, $format) = PVE::Storage::Plugin::file_size_info($path);
But this does not raise an exception if the volume does not exists? Maybe we want:
die "no such volume '$volid'\n" if !$size;
More information about the pve-devel
mailing list