Fixes an issue in which a VM fails to automatically restart after a failed stop-mode backup to pbs.
Signed-off-by: Dylan Whyte <d.wh...@proxmox.com> --- PVE/VZDump.pm | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) Notes: 1. The 1sec time delay was needed, as the check to see if the VM is running was still true while this code was executed (although the vm was just about to stop) 2. The previously used vm_status call just checks if a PID exists and returns true if so. This also returns true when the VM is in "prelauch" state, hence PVE::QemuServer::vmstatus was used to see the exact state and handle the situation accordingly. Otherwise, the VM gets stuck in prelauch state from time to time. diff --git a/PVE/VZDump.pm b/PVE/VZDump.pm index fb4c8bad..1bda1f15 100644 --- a/PVE/VZDump.pm +++ b/PVE/VZDump.pm @@ -23,6 +23,7 @@ use PVE::VZDump::Common; use PVE::VZDump::Plugin; use PVE::Tools qw(extract_param split_list); use PVE::API2Tools; +use PVE::QemuServer; my @posix_filesystems = qw(ext3 ext4 nfs nfs4 reiserfs xfs); @@ -1039,10 +1040,17 @@ sub exec_backup_task { debugmsg ('info', "resume vm", $logfd); $plugin->resume_vm ($task, $vmid); } else { - my $running = $plugin->vm_status($vmid); - if (!$running) { + sleep(1); + my $vmstatus = PVE::QemuServer::vmstatus($vmid, 1); + my $stat = $vmstatus->{$vmid}; + my $status = $stat->{qmpstatus}; + + if ($status eq "stopped") { + $plugin->start_vm ($task, $vmid); + debugmsg ('info', "restarting vm", $logfd); + } elsif ($status eq "prelaunch") { + $plugin->resume_vm ($task, $vmid); debugmsg ('info', "restarting vm", $logfd); - $plugin->start_vm ($task, $vmid); } } $self->run_hook_script ('post-restart', $task, $logfd); -- 2.20.1 _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel