From 98c8ac1a87756bc2eb601b06c72585bd869caa21 Mon Sep 17 00:00:00 2001 From: Michael Sproul Date: Tue, 31 May 2022 06:09:10 +0000 Subject: [PATCH] Fix typo in peer state transition log (#3224) ## Issue Addressed We were logging `out_finalized_epoch` instead of `our_finalized_epoch`. I noticed this ages ago but only just got around to fixing it. ## Additional Info I also reformatted the log line to respect the line length limit (`rustfmt` won't do it because it gets confused by the `;` in slog's log macros). --- beacon_node/network/src/sync/manager.rs | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/beacon_node/network/src/sync/manager.rs b/beacon_node/network/src/sync/manager.rs index 53480db88..0003db6ab 100644 --- a/beacon_node/network/src/sync/manager.rs +++ b/beacon_node/network/src/sync/manager.rs @@ -327,10 +327,17 @@ impl SyncManager { if let Some(was_updated) = update_sync_status { let is_connected = self.network_globals.peers.read().is_connected(peer_id); if was_updated { - debug!(self.log, "Peer transitioned sync state"; "peer_id" => %peer_id, "new_state" => rpr, - "our_head_slot" => local_sync_info.head_slot, "out_finalized_epoch" => local_sync_info.finalized_epoch, - "their_head_slot" => remote_sync_info.head_slot, "their_finalized_epoch" => remote_sync_info.finalized_epoch, - "is_connected" => is_connected); + debug!( + self.log, + "Peer transitioned sync state"; + "peer_id" => %peer_id, + "new_state" => rpr, + "our_head_slot" => local_sync_info.head_slot, + "our_finalized_epoch" => local_sync_info.finalized_epoch, + "their_head_slot" => remote_sync_info.head_slot, + "their_finalized_epoch" => remote_sync_info.finalized_epoch, + "is_connected" => is_connected + ); // A peer has transitioned its sync state. If the new state is "synced" we // inform the backfill sync that a new synced peer has joined us.