fix(network): Use DEBUG log level for network messages to reduce spam (…

…#12071) Fixes: #12010
near · Sep 10, 2024 · 4b11f9f · 4b11f9f
1 parent ea8ec28
commit 4b11f9f
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/chain/network/src/peer/peer_actor.rs b/chain/network/src/peer/peer_actor.rs
@@ -1403,7 +1403,7 @@ impl PeerActor {
                     } else {
                         #[cfg(test)]
                         self.network_state.config.event_sink.send(Event::RoutedMessageDropped);
-                        tracing::warn!(target: "network", ?msg, from = ?conn.peer_info.id, "Message dropped because TTL reached 0.");
+                        tracing::debug!(target: "network", ?msg, from = ?conn.peer_info.id, "Message dropped because TTL reached 0.");
                         metrics::ROUTED_MESSAGE_DROPPED
                             .with_label_values(&[msg.body_variant()])
                             .inc();
@@ -1529,7 +1529,7 @@ impl actix::Actor for PeerActor {
                 tracing::error!(target:"network", "closing reason not set. This should happen only in tests.");
             }
             Some(reason) => {
-                tracing::info!(target: "network", "{:?}: Peer {} disconnected, reason: {reason}", self.my_node_info.id, self.peer_info);
+                tracing::debug!(target: "network", "{:?}: Peer {} disconnected, reason: {reason}", self.my_node_info.id, self.peer_info);
 
                 // If we are on the inbound side of the connection, set a flag in the disconnect
                 // message advising the outbound side whether to attempt to re-establish the connection.

diff --git a/chain/network/src/peer_manager/connection/mod.rs b/chain/network/src/peer_manager/connection/mod.rs
@@ -180,7 +180,7 @@ impl Connection {
                 })
                 .await;
             if res.is_err() {
-                tracing::info!(
+                tracing::debug!(
                     "peer {} disconnected, while sending SyncAccountsData",
                     this.peer_info.id
                 );
@@ -232,7 +232,7 @@ impl Connection {
                 })
                 .await;
             if res.is_err() {
-                tracing::info!(
+                tracing::debug!(
                     "peer {} disconnected, while sending SyncSnapshotHosts",
                     this.peer_info.id
                 );

diff --git a/chain/network/src/peer_manager/network_state/mod.rs b/chain/network/src/peer_manager/network_state/mod.rs
@@ -245,7 +245,7 @@ impl NetworkState {
             peer.stop(Some(ban_reason));
         } else {
             if let Err(err) = self.peer_store.peer_ban(clock, peer_id, ban_reason) {
-                tracing::error!(target: "network", ?err, "Failed to save peer data");
+                tracing::debug!(target: "network", ?err, "Failed to save peer data");
             }
         }
     }
@@ -401,7 +401,7 @@ impl NetworkState {
                 _ => this.peer_store.peer_disconnected(&clock, &conn.peer_info.id),
             };
             if let Err(err) = res {
-                tracing::error!(target: "network", ?err, "Failed to save peer data");
+                tracing::debug!(target: "network", ?err, "Failed to save peer data");
             }
 
             // Save the fact that we are disconnecting to the ConnectionStore,