Skip to content

Commit 9c3d635

Browse files
committed
Log: pacemaker-attrd: log the cluster layer id of the changed peer
So that the messages are informative when the peer's name is unknown yet.
1 parent 4bc328b commit 9c3d635

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

daemons/attrd/attrd_corosync.c

+9-6
Original file line numberDiff line numberDiff line change
@@ -168,23 +168,26 @@ attrd_peer_change_cb(enum pcmk__node_update kind, pcmk__node_status_t *peer,
168168

169169
switch (kind) {
170170
case pcmk__node_update_name:
171-
crm_debug("%s node %s is now %s",
171+
crm_debug("%s node %s[%" PRIu32 "] is now %s",
172172
(is_remote? "Remote" : "Cluster"),
173-
peer->name, state_text(peer->state));
173+
peer->name, peer->cluster_layer_id,
174+
state_text(peer->state));
174175
break;
175176

176177
case pcmk__node_update_processes:
177178
if (!pcmk_is_set(peer->processes, crm_get_cluster_proc())) {
178179
gone = true;
179180
}
180-
crm_debug("Node %s is %s a peer",
181-
peer->name, (gone? "no longer" : "now"));
181+
crm_debug("Node %s[%" PRIu32 "] is %s a peer",
182+
peer->name, peer->cluster_layer_id,
183+
(gone? "no longer" : "now"));
182184
break;
183185

184186
case pcmk__node_update_state:
185-
crm_debug("%s node %s is now %s (was %s)",
187+
crm_debug("%s node %s[%" PRIu32 "] is now %s (was %s)",
186188
(is_remote? "Remote" : "Cluster"),
187-
peer->name, state_text(peer->state), state_text(data));
189+
peer->name, peer->cluster_layer_id,
190+
state_text(peer->state), state_text(data));
188191
if (pcmk__str_eq(peer->state, PCMK_VALUE_MEMBER, pcmk__str_none)) {
189192
/* If we're the writer, send new peers a list of all attributes
190193
* (unless it's a remote node, which doesn't run its own attrd)

0 commit comments

Comments
 (0)