From 08f8e886f181629771aad0f5bf9325bd71cb4e89 Mon Sep 17 00:00:00 2001 From: Thomas Eizinger Date: Wed, 15 Oct 2025 16:52:43 +1100 Subject: [PATCH] chore(connlib): tune down INFO logs (#10574) Several of these INFO logs are actually quite noisy, like exchanging candidates with Gateways or updating the allocation. We barely look at the INFO logs from customers and primarily investigate issues with DEBUG logs streamed to Sentry. --- rust/connlib/dns-over-tcp/src/server.rs | 2 +- rust/connlib/phoenix-channel/src/lib.rs | 2 +- rust/connlib/snownet/src/allocation.rs | 2 +- rust/connlib/snownet/src/node.rs | 8 ++++---- 4 files changed, 7 insertions(+), 7 deletions(-) diff --git a/rust/connlib/dns-over-tcp/src/server.rs b/rust/connlib/dns-over-tcp/src/server.rs index 9c06d88cc..b9ad4ba0a 100644 --- a/rust/connlib/dns-over-tcp/src/server.rs +++ b/rust/connlib/dns-over-tcp/src/server.rs @@ -98,7 +98,7 @@ impl Server { listen_endpoints.insert(handle, listen_endpoint); } - tracing::info!(%listen_endpoint, concurrency = %NUM_CONCURRENT_CLIENTS, "Created listening TCP socket"); + tracing::debug!(%listen_endpoint, concurrency = %NUM_CONCURRENT_CLIENTS, "Created listening TCP socket"); } self.sockets = sockets; diff --git a/rust/connlib/phoenix-channel/src/lib.rs b/rust/connlib/phoenix-channel/src/lib.rs index e0607356a..d2498d474 100644 --- a/rust/connlib/phoenix-channel/src/lib.rs +++ b/rust/connlib/phoenix-channel/src/lib.rs @@ -590,7 +590,7 @@ where } (Payload::Reply(Reply::Ok(OkReply::NoMessage(Empty {}))), Some(req_id)) => { if self.pending_join_requests.remove(&req_id).is_some() { - tracing::info!("Joined {} room on portal", message.topic); + tracing::debug!("Joined {} room on portal", message.topic); // For `phx_join` requests, `reply` is empty so we can safely ignore it. return Poll::Ready(Ok(Event::JoinedRoom { diff --git a/rust/connlib/snownet/src/allocation.rs b/rust/connlib/snownet/src/allocation.rs index b1c4f1f8f..f3d4c5eec 100644 --- a/rust/connlib/snownet/src/allocation.rs +++ b/rust/connlib/snownet/src/allocation.rs @@ -940,7 +940,7 @@ impl Allocation { } fn log_update(&self, now: Instant) { - tracing::info!( + tracing::debug!( host_ip4 = ?self.ip4_host_candidate.as_ref().map(|c| c.addr()), host_ip6 = ?self.ip6_host_candidate.as_ref().map(|c| c.addr()), srflx_ip4 = ?self.ip4_srflx_candidate.as_ref().map(|c| c.addr()), diff --git a/rust/connlib/snownet/src/node.rs b/rust/connlib/snownet/src/node.rs index c0b5b1be5..c997a017a 100644 --- a/rust/connlib/snownet/src/node.rs +++ b/rust/connlib/snownet/src/node.rs @@ -414,7 +414,7 @@ where return; }; - tracing::info!(?candidate, "Received candidate from remote"); + tracing::debug!(?candidate, "Received candidate from remote"); agent.add_remote_candidate(candidate.clone()); @@ -998,7 +998,7 @@ where { conn.first_handshake_completed_at = Some(now); - tracing::info!(%cid, duration_since_intent = ?conn.duration_since_intent(now), "Completed wireguard handshake"); + tracing::debug!(%cid, duration_since_intent = ?conn.duration_since_intent(now), "Completed wireguard handshake"); self.pending_events .push_back(Event::ConnectionEstablished(cid)) @@ -1311,7 +1311,7 @@ fn generate_optimistic_candidates(agent: &mut IceAgent) { .collect::>(); for c in optimistic_candidates { - tracing::info!(candidate = ?c, "Adding optimistic candidate for remote"); + tracing::debug!(candidate = ?c, "Adding optimistic candidate for remote"); agent.add_remote_candidate(c); } @@ -1343,7 +1343,7 @@ fn signal_candidate_to_remote( candidate: &Candidate, pending_events: &mut VecDeque>, ) { - tracing::info!(?candidate, "Signalling candidate to remote"); + tracing::debug!(?candidate, "Signalling candidate to remote"); pending_events.push_back(Event::NewIceCandidate { connection: id,