[pve-devel] [PATCH pve-manager] Use comma to split vmid list when building the crontab vzdump command

Dietmar Maurer dietmar at proxmox.com
Sat Sep 19 10:49:00 CEST 2015


I fixed the vzdump command, so that it accepts lists of vmids

# vzdump 100 101 102 103

So we can revert this patch now?

> On September 10, 2015 at 11:59 AM Emmanuel Kasper <e.kasper at proxmox.com>
> wrote:
> 
> 
> This fixes: https://bugzilla.proxmox.com/show_bug.cgi?id=706
> ---
>  PVE/VZDump.pm | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/PVE/VZDump.pm b/PVE/VZDump.pm
> index 138986e..78ef6e6 100644
> --- a/PVE/VZDump.pm
> +++ b/PVE/VZDump.pm
> @@ -1279,7 +1279,7 @@ sub command_line {
>      my $cmd = "vzdump";
>  
>      if ($param->{vmid}) {
> -	$cmd .= " " . join(' ', PVE::Tools::split_list($param->{vmid}));
> +		$cmd .= " " . join(',', PVE::Tools::split_list($param->{vmid}));
>      }
>  
>      foreach my $p (keys %$param) {
> -- 
> 2.1.4
> 
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel at pve.proxmox.com
> http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> 




More information about the pve-devel mailing list