proxmox-mirrors/pve-guest-common
1
0
Fork 0
mirror of https://git.proxmox.com/git/pve-guest-common synced 2025-08-15 05:06:34 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

vzdump: defaults: keep all backups by default for 7.0

Browse Source 
and switch to using prune-backups instead of maxfiles.

Storages created via the web UI defaulted to keeping all backups already, switch
to this safer default here as well.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>
This commit is contained in:
Fabian Ebner 2021-06-04 15:49:26 +02:00 committed by Fabian Grünbichler
parent a58366f460
commit 60796d5fbb
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/PVE/VZDump/Common.pm
View File

@ -210,21 +210,22 @@ my $confdesc = {
minimum => 0, minimum => 0,
default => 10, # 10 minutes default => 10, # 10 minutes
}, },
# FIXME remove with PVE 8.0 or PVE 9.0
maxfiles => { maxfiles => {
type => 'integer', type => 'integer',
description => "Maximal number of backup files per guest system.", description => "Deprecated: use 'prune-backups' instead. " .
"Maximal number of backup files per guest system.",
optional => 1, optional => 1,
minimum => 1, minimum => 1,
default => 1,
}, },
'prune-backups' => get_standard_option('prune-backups', { 'prune-backups' => get_standard_option('prune-backups', {
description => "Use these retention options instead of those from the storage configuration.", description => "Use these retention options instead of those from the storage configuration.",
optional => 1, optional => 1,
default => "keep-all=1",
}), }),
remove => { remove => {
type => 'boolean', type => 'boolean',
description => "Remove old backup files if there are more than " . description => "Prune older backups according to 'prune-backups'.",
"'maxfiles' backup files or prune according to 'prune-backups'.",
optional => 1, optional => 1,
default => 1, default => 1,
}, },