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.bek...@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;
>           }
>  
> 


_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to