vzdump: defaults: keep all backups by default for 7.0

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>

diff --git a/src/PVE/VZDump/Common.pm b/src/PVE/VZDump/Common.pm
index f32585024791d65c3f2a7127aa651c6ee630adce..83d74139e4bbb73a29b361ef8a8caacd3d4c8ee4 100644 (file)
--- a/src/PVE/VZDump/Common.pm
+++ b/src/PVE/VZDump/Common.pm
@@ -210,21 +210,22 @@ my $confdesc = {
        minimum => 0,
        default => 10, # 10 minutes
     },
+    # FIXME remove with PVE 8.0 or PVE 9.0
     maxfiles => {
        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,
        minimum => 1,
-       default => 1,
     },
     'prune-backups' => get_standard_option('prune-backups', {
        description => "Use these retention options instead of those from the storage configuration.",
        optional => 1,
+       default => "keep-all=1",
     }),
     remove => {
        type => 'boolean',
-       description => "Remove old backup files if there are more than " .
-           "'maxfiles' backup files or prune according to 'prune-backups'.",
+       description => "Prune older backups according to 'prune-backups'.",
        optional => 1,
        default => 1,
     },