Re: [pbs-devel] [PATCH] fix #3369: auto-start vms after failed pbs backup

[Dylan Whyte <d.whyte@proxmox.com>]

Please disregard this. It was sent in error to the wrong list. I will 
send an update to pve-devel shortly.

Kind regards,

Dylan

On 4/7/21 4:02 PM, Dylan Whyte wrote:
> From: dylan <dylan@pve.proxmox.com>
>
> 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.whyte@proxmox.com>
> ---
>
>   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.
>
>
>   PVE/VZDump.pm | 14 +++++++++++---
>   1 file changed, 11 insertions(+), 3 deletions(-)
>
> 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);