die if vdisk_free fail on delete_drive

replace the warn by a die.

 Currently, if we vdisk_free a disk and something goes wrong (network
 storage problem by example), the drive is removed from config and we
 cannot retry to remove it later.

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>
This commit is contained in:
Alexandre Derumier 2012-05-29 07:56:16 +02:00 committed by Dietmar Maurer
parent f5bdefa483
commit 7e4e69a6d2

View File

@ -599,7 +599,7 @@ my $delete_drive = sub {
if (&$vm_is_volid_owner($storecfg, $vmid, $volid)) {
if ($force || $key =~ m/^unused/) {
eval { PVE::Storage::vdisk_free($storecfg, $volid); };
warn $@ if $@;
die $@ if $@;
} else {
PVE::QemuServer::add_unused_volume($conf, $volid, $vmid);
}