Downgrade some less interesting info-level logs to debug (#2938)

There are a lot of these messages when Zebra starts up.
They might be slowing down CI and causing timeouts.

Co-authored-by: Janito Vaqueiro Ferreira Filho <janito.vff@gmail.com>
This commit is contained in:
teor 2021-10-22 12:11:09 +10:00 committed by GitHub
parent 424edfa4d9
commit 67327ac462
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 33 additions and 11 deletions

View File

@ -260,14 +260,34 @@ where
}
Err((addr, ref e)) => {
handshake_error_total += 1;
// this is verbose, but it's better than just hanging with no output when there are errors
info!(
?handshake_success_total,
?handshake_error_total,
?addr,
?e,
"an initial peer connection failed"
);
let mut expected_error = false;
if let Some(io_error) = e.downcast_ref::<tokio::io::Error>() {
// Some systems only have IPv4, or only have IPv6,
// so these errors are not particularly interesting.
if io_error.kind() == tokio::io::ErrorKind::AddrNotAvailable {
expected_error = true;
}
}
if expected_error {
debug!(
successes = ?handshake_success_total,
errors = ?handshake_error_total,
?addr,
?e,
"an initial peer connection failed"
);
} else {
info!(
successes = ?handshake_success_total,
errors = ?handshake_error_total,
?addr,
%e,
"an initial peer connection failed"
);
}
}
}

View File

@ -105,7 +105,7 @@ impl ConnectionTracker {
fn new(counter: &mut ActiveConnectionCounter) -> Self {
counter.count += 1;
info!(open_connections = ?counter.count, "opening a new peer connection");
debug!(open_connections = ?counter.count, "opening a new peer connection");
Self {
close_notification_tx: counter.close_notification_tx.clone(),

View File

@ -445,7 +445,7 @@ impl StateService {
let chain_tip_height = if let Some((height, _)) = self.best_tip() {
height
} else {
tracing::info!(
tracing::debug!(
response_len = ?0,
"responding to peer GetBlocks or GetHeaders with empty state",
);

View File

@ -372,7 +372,8 @@ impl Service<zn::Request> for Inbound {
.map_ok(|_resp| zn::Response::Nil)
.boxed()
} else {
info!(
// Peers send a lot of these when we first connect to them.
debug!(
"ignoring `AdvertiseTransactionIds` request from remote peer during network setup"
);
async { Ok(zn::Response::Nil) }.boxed()
@ -385,7 +386,8 @@ impl Service<zn::Request> for Inbound {
{
block_downloads.download_and_verify(hash);
} else {
info!(
// Peers send a lot of these when we first connect to them.
debug!(
?hash,
"ignoring `AdvertiseBlock` request from remote peer during network setup"
);