[pve-devel] [PATCH ha-manager 1/2] fix some log formatting
Thomas Lamprecht
t.lamprecht at proxmox.com
Tue Feb 16 15:33:55 CET 2016
First remove trailing whitespace from log messages on state changes
This needs to touch some regression test, but with no change in
semantics.
Second add a missing paranthese on the "fixup service location"
message. This needs no regression test log.expect changes.
Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
---
Even if this touches some log.expect lines it's better now
then later with more tests.
src/PVE/HA/Manager.pm | 8 ++++----
src/test/test-basic1/log.expect | 6 +++---
src/test/test-basic5/log.expect | 2 +-
src/test/test-resource-failure1/log.expect | 4 ++--
src/test/test-resource-failure2/log.expect | 4 ++--
src/test/test-resource-failure4/log.expect | 6 +++---
src/test/test-resource-failure5/log.expect | 8 ++++----
src/test/test-restart-lrm/log.expect | 4 ++--
src/test/test-service-command1/log.expect | 4 ++--
src/test/test-shutdown1/log.expect | 2 +-
src/test/test-shutdown2/log.expect | 2 +-
src/test/test-shutdown3/log.expect | 2 +-
src/test/test-shutdown4/log.expect | 2 +-
13 files changed, 27 insertions(+), 27 deletions(-)
diff --git a/src/PVE/HA/Manager.pm b/src/PVE/HA/Manager.pm
index cab2b0a..a482ef2 100644
--- a/src/PVE/HA/Manager.pm
+++ b/src/PVE/HA/Manager.pm
@@ -207,10 +207,10 @@ my $change_service_state = sub {
$self->recompute_online_node_usage();
$sd->{uid} = compute_new_uuid($new_state);
-
- $text_state = " ($text_state)" if $text_state;
- $haenv->log('info', "service '$sid': state changed from '${old_state}' to '${new_state}' $text_state");
+ $text_state = " ($text_state)" if $text_state;
+ $haenv->log('info', "service '$sid': state changed from '${old_state}'" .
+ " to '${new_state}'$text_state");
};
# after a node was fenced this recovers the service to a new node
@@ -495,7 +495,7 @@ sub next_state_stopped {
if ($sd->{node} ne $cd->{node}) {
# this can happen if we fence a node with active migrations
# hack: modify $sd (normally this should be considered read-only)
- $haenv->log('info', "fixup service '$sid' location ($sd->{node} => $cd->{node}");
+ $haenv->log('info', "fixup service '$sid' location ($sd->{node} => $cd->{node})");
$sd->{node} = $cd->{node};
}
diff --git a/src/test/test-basic1/log.expect b/src/test/test-basic1/log.expect
index 5cd30f0..68df71b 100644
--- a/src/test/test-basic1/log.expect
+++ b/src/test/test-basic1/log.expect
@@ -16,7 +16,7 @@ info 20 node1/crm: node 'node3': state changed from 'unknown' => 'online'
info 20 node1/crm: adding new service 'vm:101' on node 'node1'
info 20 node1/crm: adding new service 'vm:102' on node 'node2'
info 20 node1/crm: adding new service 'vm:103' on node 'node3'
-info 20 node1/crm: service 'vm:102': state changed from 'started' to 'request_stop'
+info 20 node1/crm: service 'vm:102': state changed from 'started' to 'request_stop'
info 21 node1/lrm: got lock 'ha_agent_node1_lock'
info 21 node1/lrm: status change wait_for_agent_lock => active
info 21 node1/lrm: starting service vm:101
@@ -29,12 +29,12 @@ info 25 node3/lrm: got lock 'ha_agent_node3_lock'
info 25 node3/lrm: status change wait_for_agent_lock => active
info 25 node3/lrm: starting service vm:103
info 25 node3/lrm: service status vm:103 started
-info 40 node1/crm: service 'vm:102': state changed from 'request_stop' to 'stopped'
+info 40 node1/crm: service 'vm:102': state changed from 'request_stop' to 'stopped'
info 120 cmdlist: execute network node3 off
info 120 node1/crm: node 'node3': state changed from 'online' => 'unknown'
info 124 node3/crm: status change slave => wait_for_quorum
info 125 node3/lrm: status change active => lost_agent_lock
-info 160 node1/crm: service 'vm:103': state changed from 'started' to 'fence'
+info 160 node1/crm: service 'vm:103': state changed from 'started' to 'fence'
info 160 node1/crm: node 'node3': state changed from 'unknown' => 'fence'
info 166 watchdog: execute power node3 off
info 165 node3/crm: killed by poweroff
diff --git a/src/test/test-basic5/log.expect b/src/test/test-basic5/log.expect
index e640e13..54b579c 100644
--- a/src/test/test-basic5/log.expect
+++ b/src/test/test-basic5/log.expect
@@ -41,7 +41,7 @@ info 162 hardware: server 'node1' stopped by poweroff (watchdog)
info 222 node3/crm: got lock 'ha_manager_lock'
info 222 node3/crm: status change slave => master
info 222 node3/crm: node 'node1': state changed from 'online' => 'unknown'
-info 282 node3/crm: service 'vm:101': state changed from 'started' to 'fence'
+info 282 node3/crm: service 'vm:101': state changed from 'started' to 'fence'
info 282 node3/crm: node 'node1': state changed from 'unknown' => 'fence'
info 282 node3/crm: got lock 'ha_agent_node1_lock'
info 282 node3/crm: fencing: acknowleged - got agent lock for node 'node1'
diff --git a/src/test/test-resource-failure1/log.expect b/src/test/test-resource-failure1/log.expect
index 97d9fdb..c3170fc 100644
--- a/src/test/test-resource-failure1/log.expect
+++ b/src/test/test-resource-failure1/log.expect
@@ -14,12 +14,12 @@ info 20 node1/crm: node 'node1': state changed from 'unknown' => 'online'
info 20 node1/crm: node 'node2': state changed from 'unknown' => 'online'
info 20 node1/crm: node 'node3': state changed from 'unknown' => 'online'
info 20 node1/crm: adding new service 'fa:110' on node 'node2'
-info 20 node1/crm: service 'fa:110': state changed from 'started' to 'request_stop'
+info 20 node1/crm: service 'fa:110': state changed from 'started' to 'request_stop'
info 22 node2/crm: status change wait_for_quorum => slave
info 23 node2/lrm: got lock 'ha_agent_node2_lock'
info 23 node2/lrm: status change wait_for_agent_lock => active
info 24 node3/crm: status change wait_for_quorum => slave
-info 40 node1/crm: service 'fa:110': state changed from 'request_stop' to 'stopped'
+info 40 node1/crm: service 'fa:110': state changed from 'request_stop' to 'stopped'
info 120 cmdlist: execute service fa:110 enabled
info 120 node1/crm: service 'fa:110': state changed from 'stopped' to 'started' (node = node2)
info 123 node2/lrm: starting service fa:110
diff --git a/src/test/test-resource-failure2/log.expect b/src/test/test-resource-failure2/log.expect
index 3c827f8..604ad95 100644
--- a/src/test/test-resource-failure2/log.expect
+++ b/src/test/test-resource-failure2/log.expect
@@ -14,12 +14,12 @@ info 20 node1/crm: node 'node1': state changed from 'unknown' => 'online'
info 20 node1/crm: node 'node2': state changed from 'unknown' => 'online'
info 20 node1/crm: node 'node3': state changed from 'unknown' => 'online'
info 20 node1/crm: adding new service 'fa:130' on node 'node2'
-info 20 node1/crm: service 'fa:130': state changed from 'started' to 'request_stop'
+info 20 node1/crm: service 'fa:130': state changed from 'started' to 'request_stop'
info 22 node2/crm: status change wait_for_quorum => slave
info 23 node2/lrm: got lock 'ha_agent_node2_lock'
info 23 node2/lrm: status change wait_for_agent_lock => active
info 24 node3/crm: status change wait_for_quorum => slave
-info 40 node1/crm: service 'fa:130': state changed from 'request_stop' to 'stopped'
+info 40 node1/crm: service 'fa:130': state changed from 'request_stop' to 'stopped'
info 120 cmdlist: execute service fa:130 enabled
info 120 node1/crm: service 'fa:130': state changed from 'stopped' to 'started' (node = node2)
info 123 node2/lrm: starting service fa:130
diff --git a/src/test/test-resource-failure4/log.expect b/src/test/test-resource-failure4/log.expect
index 3e8eef9..77c2367 100644
--- a/src/test/test-resource-failure4/log.expect
+++ b/src/test/test-resource-failure4/log.expect
@@ -21,12 +21,12 @@ info 25 node3/lrm: status change wait_for_agent_lock => active
info 25 node3/lrm: starting service fa:1001
info 25 node3/lrm: service status fa:1001 started
info 120 cmdlist: execute service fa:1001 disabled
-info 120 node1/crm: service 'fa:1001': state changed from 'started' to 'request_stop'
+info 120 node1/crm: service 'fa:1001': state changed from 'started' to 'request_stop'
info 125 node3/lrm: stopping service fa:1001
info 125 node3/lrm: unable to stop stop service fa:1001 (still running)
err 140 node1/crm: service 'fa:1001' stop failed (exit code 1)
-info 140 node1/crm: service 'fa:1001': state changed from 'request_stop' to 'error'
-info 140 node1/crm: service 'fa:1001': state changed from 'error' to 'stopped'
+info 140 node1/crm: service 'fa:1001': state changed from 'request_stop' to 'error'
+info 140 node1/crm: service 'fa:1001': state changed from 'error' to 'stopped'
info 145 node3/lrm: stopping service fa:1001
info 145 node3/lrm: unable to stop stop service fa:1001 (still running)
info 165 node3/lrm: stopping service fa:1001
diff --git a/src/test/test-resource-failure5/log.expect b/src/test/test-resource-failure5/log.expect
index f17421a..b6e7807 100644
--- a/src/test/test-resource-failure5/log.expect
+++ b/src/test/test-resource-failure5/log.expect
@@ -14,12 +14,12 @@ info 20 node1/crm: node 'node1': state changed from 'unknown' => 'online'
info 20 node1/crm: node 'node2': state changed from 'unknown' => 'online'
info 20 node1/crm: node 'node3': state changed from 'unknown' => 'online'
info 20 node1/crm: adding new service 'fa:130' on node 'node2'
-info 20 node1/crm: service 'fa:130': state changed from 'started' to 'request_stop'
+info 20 node1/crm: service 'fa:130': state changed from 'started' to 'request_stop'
info 22 node2/crm: status change wait_for_quorum => slave
info 23 node2/lrm: got lock 'ha_agent_node2_lock'
info 23 node2/lrm: status change wait_for_agent_lock => active
info 24 node3/crm: status change wait_for_quorum => slave
-info 40 node1/crm: service 'fa:130': state changed from 'request_stop' to 'stopped'
+info 40 node1/crm: service 'fa:130': state changed from 'request_stop' to 'stopped'
info 120 cmdlist: execute service fa:130 enabled
info 120 node1/crm: service 'fa:130': state changed from 'stopped' to 'started' (node = node2)
info 123 node2/lrm: starting service fa:130
@@ -29,10 +29,10 @@ info 143 node2/lrm: starting service fa:130
warn 143 node2/lrm: unable to start service fa:130
err 143 node2/lrm: unable to start service fa:130 on local node after 1 retries
err 160 node1/crm: recovery policy for service fa:130 failed, entering error state!
-info 160 node1/crm: service 'fa:130': state changed from 'started' to 'error'
+info 160 node1/crm: service 'fa:130': state changed from 'started' to 'error'
warn 163 node2/lrm: service fa:130 is not running and in an error state
warn 183 node2/lrm: service fa:130 is not running and in an error state
warn 203 node2/lrm: service fa:130 is not running and in an error state
info 220 cmdlist: execute service fa:130 disabled
-info 220 node1/crm: service 'fa:130': state changed from 'error' to 'stopped'
+info 220 node1/crm: service 'fa:130': state changed from 'error' to 'stopped'
info 820 hardware: exit simulation - done
diff --git a/src/test/test-restart-lrm/log.expect b/src/test/test-restart-lrm/log.expect
index 7e0a71e..198aef9 100644
--- a/src/test/test-restart-lrm/log.expect
+++ b/src/test/test-restart-lrm/log.expect
@@ -22,10 +22,10 @@ info 25 node3/lrm: starting service vm:103
info 25 node3/lrm: service status vm:103 started
info 120 cmdlist: execute restart-lrm node3
info 120 node3/lrm: restart LRM, freeze all services
-info 120 node1/crm: service 'vm:103': state changed from 'started' to 'freeze'
+info 120 node1/crm: service 'vm:103': state changed from 'started' to 'freeze'
info 126 node3/lrm: exit (loop end)
info 126 node3/lrm: status change startup => wait_for_agent_lock
-info 160 node1/crm: service 'vm:103': state changed from 'freeze' to 'started'
+info 160 node1/crm: service 'vm:103': state changed from 'freeze' to 'started'
info 165 node3/lrm: got lock 'ha_agent_node3_lock'
info 165 node3/lrm: status change wait_for_agent_lock => active
info 720 hardware: exit simulation - done
diff --git a/src/test/test-service-command1/log.expect b/src/test/test-service-command1/log.expect
index 8e2d1f1..7e74e8a 100644
--- a/src/test/test-service-command1/log.expect
+++ b/src/test/test-service-command1/log.expect
@@ -21,10 +21,10 @@ info 25 node3/lrm: status change wait_for_agent_lock => active
info 25 node3/lrm: starting service vm:103
info 25 node3/lrm: service status vm:103 started
info 120 cmdlist: execute service vm:103 disabled
-info 120 node1/crm: service 'vm:103': state changed from 'started' to 'request_stop'
+info 120 node1/crm: service 'vm:103': state changed from 'started' to 'request_stop'
info 125 node3/lrm: stopping service vm:103
info 125 node3/lrm: service status vm:103 stopped
-info 140 node1/crm: service 'vm:103': state changed from 'request_stop' to 'stopped'
+info 140 node1/crm: service 'vm:103': state changed from 'request_stop' to 'stopped'
info 220 cmdlist: execute service vm:103 enabled
info 220 node1/crm: service 'vm:103': state changed from 'stopped' to 'started' (node = node3)
info 225 node3/lrm: starting service vm:103
diff --git a/src/test/test-shutdown1/log.expect b/src/test/test-shutdown1/log.expect
index 95937cb..9dbdb84 100644
--- a/src/test/test-shutdown1/log.expect
+++ b/src/test/test-shutdown1/log.expect
@@ -29,7 +29,7 @@ info 126 shutdown: execute crm node3 stop
info 145 node3/crm: exit (loop end)
info 145 shutdown: execute power node3 off
info 160 node1/crm: node 'node3': state changed from 'online' => 'unknown'
-info 200 node1/crm: service 'vm:103': state changed from 'started' to 'fence'
+info 200 node1/crm: service 'vm:103': state changed from 'started' to 'fence'
info 200 node1/crm: node 'node3': state changed from 'unknown' => 'fence'
info 200 node1/crm: got lock 'ha_agent_node3_lock'
info 200 node1/crm: fencing: acknowleged - got agent lock for node 'node3'
diff --git a/src/test/test-shutdown2/log.expect b/src/test/test-shutdown2/log.expect
index fb959f9..c3fbb07 100644
--- a/src/test/test-shutdown2/log.expect
+++ b/src/test/test-shutdown2/log.expect
@@ -29,7 +29,7 @@ info 126 shutdown: execute crm node3 stop
info 145 node3/crm: exit (loop end)
info 145 shutdown: execute power node3 off
info 160 node1/crm: node 'node3': state changed from 'online' => 'unknown'
-info 200 node1/crm: service 'vm:103': state changed from 'started' to 'fence'
+info 200 node1/crm: service 'vm:103': state changed from 'started' to 'fence'
info 200 node1/crm: node 'node3': state changed from 'unknown' => 'fence'
info 200 node1/crm: got lock 'ha_agent_node3_lock'
info 200 node1/crm: fencing: acknowleged - got agent lock for node 'node3'
diff --git a/src/test/test-shutdown3/log.expect b/src/test/test-shutdown3/log.expect
index 4efa3e7..16d8c4e 100644
--- a/src/test/test-shutdown3/log.expect
+++ b/src/test/test-shutdown3/log.expect
@@ -29,7 +29,7 @@ info 126 shutdown: execute crm node3 stop
info 145 node3/crm: exit (loop end)
info 145 shutdown: execute power node3 off
info 160 node1/crm: node 'node3': state changed from 'online' => 'unknown'
-info 200 node1/crm: service 'ct:103': state changed from 'started' to 'fence'
+info 200 node1/crm: service 'ct:103': state changed from 'started' to 'fence'
info 200 node1/crm: node 'node3': state changed from 'unknown' => 'fence'
info 200 node1/crm: got lock 'ha_agent_node3_lock'
info 200 node1/crm: fencing: acknowleged - got agent lock for node 'node3'
diff --git a/src/test/test-shutdown4/log.expect b/src/test/test-shutdown4/log.expect
index aa8cfcc..843104b 100644
--- a/src/test/test-shutdown4/log.expect
+++ b/src/test/test-shutdown4/log.expect
@@ -32,7 +32,7 @@ info 141 shutdown: execute power node1 off
info 141 node2/crm: got lock 'ha_manager_lock'
info 141 node2/crm: status change slave => master
info 141 node2/crm: node 'node1': state changed from 'online' => 'unknown'
-info 220 node2/crm: service 'vm:100': state changed from 'started' to 'fence'
+info 220 node2/crm: service 'vm:100': state changed from 'started' to 'fence'
info 220 node2/crm: node 'node1': state changed from 'unknown' => 'fence'
info 220 node2/crm: got lock 'ha_agent_node1_lock'
info 220 node2/crm: fencing: acknowleged - got agent lock for node 'node1'
--
2.1.4
More information about the pve-devel
mailing list