more logging improvements (#4885)

## Issue Addressed -downgrades `Missing components over rpc` to debug because this isn't unusual and just results in a re-try -removes the result from `Block component processed for lookup` because this prints the full block on an unknown parent error Co-authored-by: realbigsean <[email protected]>
sigp · Nov 3, 2023 · bcca88a · bcca88a
1 parent f2aabe9
commit bcca88a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/beacon_node/network/src/network_beacon_processor/sync_methods.rs b/beacon_node/network/src/network_beacon_processor/sync_methods.rs
@@ -340,7 +340,7 @@ impl<T: BeaconChainTypes> NetworkBeaconProcessor<T> {
                 );
             }
             Ok(AvailabilityProcessingStatus::MissingComponents(_, _)) => {
-                warn!(
+                debug!(
                     self.log,
                     "Missing components over rpc";
                     "block_hash" => %block_root,

diff --git a/beacon_node/network/src/sync/block_lookups/mod.rs b/beacon_node/network/src/sync/block_lookups/mod.rs
@@ -786,7 +786,7 @@ impl<T: BeaconChainTypes> BlockLookups<T> {
             self.log,
             "Block component processed for lookup";
             "response_type" => ?R::response_type(),
-            "result" => ?result,
+            "block_root" => ?root,
         );
 
         match result {