[pve-devel] [PATCH stable5 manager 5/6] 5to6: add more corosync subheaders

Fabian Grünbichler f.gruenbichler at proxmox.com
Mon Jul 8 13:54:08 CEST 2019


to improve readability

Signed-off-by: Fabian Grünbichler <f.gruenbichler at proxmox.com>
(cherry picked from commit 5684da54dc15fb2a5bb26fdef95db67cea836a21)
---
 PVE/CLI/pve5to6.pm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/PVE/CLI/pve5to6.pm b/PVE/CLI/pve5to6.pm
index e7c9a24f..0c7efbb1 100644
--- a/PVE/CLI/pve5to6.pm
+++ b/PVE/CLI/pve5to6.pm
@@ -365,6 +365,7 @@ sub check_cluster_corosync {
     my $conf_nodelist = PVE::Corosync::nodelist($conf);
     my $node_votes = 0;
 
+    print "\nAnalzying quorum settings and state..\n";
     if (!defined($conf_nodelist)) {
 	log_fail("unable to retrieve nodelist from corosync.conf");
     } else {
@@ -424,6 +425,7 @@ sub check_cluster_corosync {
     log_fail("corosync.conf ($conf_nodelist_count) and pmxcfs ($cfs_nodelist_count) don't agree about size of nodelist.")
 	if $conf_nodelist_count != $cfs_nodelist_count;
 
+    print "\nChecking nodelist entries..\n";
     foreach my $cs_node (keys %$conf_nodelist) {
 	my $entry = $conf_nodelist->{$cs_node};
 	log_fail("$cs_node: no name entry in corosync.conf.")
@@ -453,8 +455,8 @@ sub check_cluster_corosync {
 	$verify_ring_ip->('ring1_addr');
     }
 
+    print "\nChecking totem settings..\n";
     my $totem = $conf->{main}->{totem};
-
     my $transport = $totem->{transport};
     if (defined($transport)) {
 	log_fail("Corosync transport explicitly set to '$transport' instead of implicit default!");
-- 
2.20.1





More information about the pve-devel mailing list