5to6: make corosync totem checks more verbose

to avoid just printing the subheader with no results

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

diff --git a/PVE/CLI/pve5to6.pm b/PVE/CLI/pve5to6.pm
index edcddaaaac90382ef3367917348b9891c8f68572..2724e33d0e826e832850cb6b350d0e5a2ba74df8 100644 (file)
--- a/PVE/CLI/pve5to6.pm
+++ b/PVE/CLI/pve5to6.pm
@@ -395,15 +395,23 @@ sub check_cluster_corosync {
     my $totem = $conf->{main}->{totem};
     my $transport = $totem->{transport};
     if (defined($transport)) {
-       log_fail("Corosync transport explicitly set to '$transport' instead of implicit default!");
+       if ($transport ne 'knet') {
+           log_fail("Corosync transport explicitly set to '$transport' instead of implicit default!");
+       } else {
+           log_pass("Corosync transport set to '$transport'.");
+       }
+    } else {
+       log_pass("Corosync transport set to implicit default.");
     }
 
     if ((!defined($totem->{secauth}) || $totem->{secauth} ne 'on') && (!defined($totem->{crypto_cipher}) || $totem->{crypto_cipher} eq 'none')) {
        log_fail("Corosync authentication/encryption is not explicitly enabled (secauth / crypto_cipher / crypto_hash)!");
-    }
-
-    if (defined($totem->{crypto_cipher}) && $totem->{crypto_cipher} eq '3des') {
-       log_fail("Corosync encryption cipher set to '3des', no longer supported in Corosync 3.x!");
+    } else {
+       if (defined($totem->{crypto_cipher}) && $totem->{crypto_cipher} eq '3des') {
+           log_fail("Corosync encryption cipher set to '3des', no longer supported in Corosync 3.x!");
+       } else {
+           log_pass("Corosync encryption and authentication enabled.");
+       }
     }
 
     print "\n";