From 8d2939700da85ad8fa651ca390aaba57f65e76d1 Mon Sep 17 00:00:00 2001
From: Dmitry Markin <dmitry@markin.tech>
Date: Thu, 16 May 2024 15:46:19 +0300
Subject: [PATCH] Demote per-peer validation slots warning to debug (#4480)
Demote `Ignored block announcement because all validation slots for this
peer are occupied.` message to debug level.
This is mostly an indicator of somebody spamming the node or (more
likely) some node actively keeping up with the network but not
recognizing it's in a major sync mode, so sending zillions of block
announcements (have seen this on Versi).
This warning shouldn't be considered an error by the end user, so let's
make it debug.
Ref. https://github.com/paritytech/polkadot-sdk/issues/1929.
---
substrate/client/network/sync/src/block_announce_validator.rs | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/substrate/client/network/sync/src/block_announce_validator.rs b/substrate/client/network/sync/src/block_announce_validator.rs
index 3c994dd6944..cb1d5ee6b22 100644
--- a/substrate/client/network/sync/src/block_announce_validator.rs
+++ b/substrate/client/network/sync/src/block_announce_validator.rs
@@ -156,7 +156,7 @@ impl<B: BlockT> BlockAnnounceValidator<B> {
return
},
AllocateSlotForBlockAnnounceValidation::MaximumPeerSlotsReached => {
- warn!(
+ debug!(
target: LOG_TARGET,
"💔 Ignored block (#{} -- {}) announcement from {} because all validation slots for this peer are occupied.",
number,
--
GitLab