]> git.proxmox.com Git - pve-manager.git/commitdiff
pve7to8: fix Ceph noout check
authorFiona Ebner <f.ebner@proxmox.com>
Mon, 17 Jul 2023 08:38:19 +0000 (10:38 +0200)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Sat, 15 Jul 2023 21:28:46 +0000 (23:28 +0200)
Commit 114e5f2c ("pve7to8: sync over from stable-7 branch")
accidentally got rid of the correct value 0 here and also the new TODO
message to improve the situation. But the TODO is actually easy,
because there already is the $upgraded variable. Just rely on that
instead of hard-coding and forgetting about it again.

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
PVE/CLI/pve7to8.pm

index c7d3f19c132c4f7cdcc147ed18463931ae1c76ab..5ba738372d0a4eaeaa6ab57e27c701797af53996 100644 (file)
@@ -548,7 +548,7 @@ sub check_ceph {
            log_warn("unable to determine overall Ceph daemon versions!");
        } elsif (keys %$overall_versions == 1) {
            log_pass("single running overall version detected for all Ceph daemon types.");
-           $noout_wanted = 1; # off post-upgrade, on pre-upgrade
+           $noout_wanted = !$upgraded; # off post-upgrade, on pre-upgrade
        } elsif (keys $ceph_versions_simple->{overall}->%* != 1) {
            log_warn("overall version mismatch detected, check 'ceph versions' output for details!");
        }