[pve-devel] applied: [PATCH v2 manager] change error message related to maxbackups

[Thomas Lamprecht]

On 6/11/19 5:55 PM, Oguz Bektas wrote:
> make the message a bit more informative (with help from thomas), namely
> mentioning the ability to change/increase the limit.
> 
> Signed-off-by: Oguz Bektas <o.bektas at proxmox.com>
> ---
> 
> v1 -> v2:
> * change wording with suggestion from thomas
> 
>  PVE/VZDump.pm | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/PVE/VZDump.pm b/PVE/VZDump.pm
> index 0a35eea7..7f5da90e 100644
> --- a/PVE/VZDump.pm
> +++ b/PVE/VZDump.pm
> @@ -780,7 +780,9 @@ sub exec_backup_task {
>  
>  	if ($maxfiles && !$opts->{remove}) {
>  	    my $bklist = get_backup_file_list($opts->{dumpdir}, $bkname);
> -	    die "only $maxfiles backup(s) allowed - please consider to remove old backup files.\n" 
> +	    die "There is a max backup limit of ($maxfiles) enforced by the".
> +	    " target storage or the vzdump parameters.".
> +	    " Either increase the limit or delete old backup(s).\n"
>  		if scalar(@$bklist) >= $maxfiles;
>  	}
>  
> 

applied, thanks!