[pve-devel] [PATCH manager] fix #2598: prepare/activate volumes after stopping container for backup

[Thomas Lamprecht]

On 2/18/20 1:35 PM, Oguz Bektas wrote:
> when doing a 'stop' backup with an LVM backend, volumes are deactivated
> by the stop operation. they're not activated before the backup, which
> causes it to fail because of mount/unmount problems.
> 
> call prepare() after stop_vm() instead to activate volumes beforehand.
> 

and that's all right? I do not think this is the correct solution,
we not just do the activate volume in prepare..

> [0]: https://forum.proxmox.com/threads/problem-backups-proxmox-6-1.65317/
> 
> Signed-off-by: Oguz Bektas <o.bektas at proxmox.com>
> ---
>  PVE/VZDump.pm | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/PVE/VZDump.pm b/PVE/VZDump.pm
> index 87d4b699..514f432b 100644
> --- a/PVE/VZDump.pm
> +++ b/PVE/VZDump.pm
> @@ -757,7 +757,6 @@ sub exec_backup_task {
>  
>  	if ($mode eq 'stop') {
>  
> -	    $plugin->prepare ($task, $vmid, $mode);
>  
>  	    $self->run_hook_script ('backup-start', $task, $logfd);
>  
> @@ -766,6 +765,7 @@ sub exec_backup_task {
>  		$task->{vmstoptime} = time();
>  		$self->run_hook_script ('pre-stop', $task, $logfd);
>  		$plugin->stop_vm ($task, $vmid);
> +		$plugin->prepare ($task, $vmid, $mode);
>  		$cleanup->{restart} = 1;
>  	    }
>  
>