mirror of
https://git.proxmox.com/git/qemu-server
synced 2025-05-03 03:38:29 +00:00
api: snapshot delete: prefix lock-fail error
otherwise users might get confused if they just get a message about a migrate lock not being available.. Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
This commit is contained in:
parent
fdbbed2fac
commit
1770b70fdf
@ -4580,7 +4580,9 @@ __PACKAGE__->register_method({
|
|||||||
|
|
||||||
my $snapname = extract_param($param, 'snapname');
|
my $snapname = extract_param($param, 'snapname');
|
||||||
|
|
||||||
|
my $lock_obtained;
|
||||||
my $do_delete = sub {
|
my $do_delete = sub {
|
||||||
|
$lock_obtained = 1;
|
||||||
PVE::Cluster::log_msg('info', $authuser, "delete snapshot VM $vmid: $snapname");
|
PVE::Cluster::log_msg('info', $authuser, "delete snapshot VM $vmid: $snapname");
|
||||||
PVE::QemuConfig->snapshot_delete($vmid, $snapname, $param->{force});
|
PVE::QemuConfig->snapshot_delete($vmid, $snapname, $param->{force});
|
||||||
};
|
};
|
||||||
@ -4589,7 +4591,11 @@ __PACKAGE__->register_method({
|
|||||||
if ($param->{force}) {
|
if ($param->{force}) {
|
||||||
$do_delete->();
|
$do_delete->();
|
||||||
} else {
|
} else {
|
||||||
PVE::GuestHelpers::guest_migration_lock($vmid, 10, $do_delete);
|
eval { PVE::GuestHelpers::guest_migration_lock($vmid, 10, $do_delete); };
|
||||||
|
if (my $err = $@) {
|
||||||
|
die $err if $lock_obtained;
|
||||||
|
die "Failed to obtain guest migration lock - replication running?\n";
|
||||||
|
}
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user