pve7to8: remove outdated warning about retention

It just talks about the default behavior since PVE 7. It's rather
confusing to mention this, because the behavior doesn't change anymore
in PVE 8.

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
(cherry picked from commit b58348b1b60c6546b14323ec9303c91d5b1a29b7)
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

diff --git a/PVE/CLI/pve7to8.pm b/PVE/CLI/pve7to8.pm
index 27cef7841f0ad46ca905a08546eddde439e6ffdd..c083b95e23744f3a8f88e8d07bd13e7c969dbd14 100644 (file)
--- a/PVE/CLI/pve7to8.pm
+++ b/PVE/CLI/pve7to8.pm
@@ -609,8 +609,6 @@ sub check_backup_retention_settings {
 
     my $pass = 1;
 
-    my $node_has_retention;
-
     my $maxfiles_msg = "parameter 'maxfiles' is deprecated with PVE 7.x and will be removed in a " .
        "future version, use 'prune-backups' instead.";
 
@@ -627,8 +625,6 @@ sub check_backup_retention_settings {
            $pass = 0;
            log_warn("$fn - $maxfiles_msg");
        }
-
-       $node_has_retention = defined($param->{maxfiles}) || defined($param->{'prune-backups'});
     };
     if (my $err = $@) {
        $pass = 0;
@@ -644,15 +640,6 @@ sub check_backup_retention_settings {
            $pass = 0;
            log_warn("storage '$storeid' - $maxfiles_msg");
        }
-
-       next if !$scfg->{content}->{backup};
-       next if defined($scfg->{maxfiles}) || defined($scfg->{'prune-backups'});
-       next if $node_has_retention;
-
-       log_info(
-           "storage '$storeid' - no backup retention settings defined - by default, since PVE 7.0"
-           ." it will no longer keep only the last backup, but all backups"
-       );
     }
 
     eval {