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

archive info: include archive name in error message

Browse Source 
so that we have some context if users report issues with it..

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
This commit is contained in:
Thomas Lamprecht
2020-05-15 19:13:37 +02:00
parent c265d42160
commit bf5af0fbf5
2 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
PVE/Storage.pm
View File

@ -1402,7 +1402,7 @@ sub archive_info {
$info->{type} = $1;
}
} else {
die "ERROR: couldn't determine format and compression type\n";
die "ERROR: couldn't determine archive info from '$archive'\n";
}
return $info;

5
test/archive_info_test.pm
View File

@ -100,10 +100,11 @@ my $non_bkp_suffix = {
for my $virt (sort keys %$non_bkp_suffix) {
my $suffix = $non_bkp_suffix->{$virt};
for my $s (@$suffix) {
my $archive = "backup/vzdump-$virt-$vmid-2020_03_30-21_12_40.$s";
push @$tests, {
description => "Failed match: Backup archive, $virt, $s",
archive => "backup/vzdump-$virt-$vmid-2020_03_30-21_12_40.$s",
expected => "ERROR: couldn't determine format and compression type\n",
archive => $archive,
expected => "ERROR: couldn't determine archive info from '$archive'\n",
};
}
}