From 2af9c9aa6e35bb1f3cc87d61a2abb06c28341ad5 Mon Sep 17 00:00:00 2001 From: Senthilkumar Gopal <787381+sengopal@users.noreply.github.com> Date: Fri, 17 Apr 2026 09:36:47 -0700 Subject: [PATCH 1/3] fix(runtime): downgrade per-request lifecycle logs from info to debug "request received" and "request completed" fire on every inbound request, producing excessive log noise at default log levels. Debug is the correct level for per-request chatter; info should be reserved for meaningful state changes. Co-Authored-By: Claude Sonnet 4.6 --- lib/runtime/src/pipeline/network/ingress/push_handler.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/runtime/src/pipeline/network/ingress/push_handler.rs b/lib/runtime/src/pipeline/network/ingress/push_handler.rs index dfbc69720c0a..6f37c44e2820 100644 --- a/lib/runtime/src/pipeline/network/ingress/push_handler.rs +++ b/lib/runtime/src/pipeline/network/ingress/push_handler.rs @@ -133,7 +133,7 @@ impl Drop for RequestMetricsGuard { self.request_duration .observe(self.start_time.elapsed().as_secs_f64()); if let Some(request_id) = &self.request_id { - tracing::info!(request_id = %request_id, "request completed"); + tracing::debug!(request_id = %request_id, "request completed"); } } } @@ -179,7 +179,7 @@ where m.inflight_requests.inc(); m.request_bytes.inc_by(payload.len() as u64); if let Some(rid) = &request_id { - tracing::info!(request_id = %rid, "request received"); + tracing::debug!(request_id = %rid, "request received"); } RequestMetricsGuard { inflight_requests: m.inflight_requests.clone(), From 1b4218bcace0318be16552c28aa7357cf5ba4ccb Mon Sep 17 00:00:00 2001 From: Senthilkumar Gopal <787381+sengopal@users.noreply.github.com> Date: Fri, 17 Apr 2026 09:49:09 -0700 Subject: [PATCH 2/3] fix(runtime): remove duplicate request_id from lifecycle log fields The handle_payload span already carries request_id, component, endpoint, namespace, and instance_id. Explicitly repeating request_id in the info! call caused it to appear twice in log output. Drop the redundant field so all structured context comes from the span once. Co-Authored-By: Claude Sonnet 4.6 --- lib/runtime/src/pipeline/network/ingress/push_handler.rs | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/runtime/src/pipeline/network/ingress/push_handler.rs b/lib/runtime/src/pipeline/network/ingress/push_handler.rs index 6f37c44e2820..146b837a3590 100644 --- a/lib/runtime/src/pipeline/network/ingress/push_handler.rs +++ b/lib/runtime/src/pipeline/network/ingress/push_handler.rs @@ -132,8 +132,8 @@ impl Drop for RequestMetricsGuard { self.inflight_requests.dec(); self.request_duration .observe(self.start_time.elapsed().as_secs_f64()); - if let Some(request_id) = &self.request_id { - tracing::debug!(request_id = %request_id, "request completed"); + if self.request_id.is_some() { + tracing::info!("request completed"); } } } @@ -178,8 +178,8 @@ where m.request_counter.inc(); m.inflight_requests.inc(); m.request_bytes.inc_by(payload.len() as u64); - if let Some(rid) = &request_id { - tracing::debug!(request_id = %rid, "request received"); + if request_id.is_some() { + tracing::info!("request received"); } RequestMetricsGuard { inflight_requests: m.inflight_requests.clone(), From aad4fccce58021c3519a2d5c4cc493d8ceec948e Mon Sep 17 00:00:00 2001 From: Senthilkumar Gopal <787381+sengopal@users.noreply.github.com> Date: Fri, 17 Apr 2026 09:53:15 -0700 Subject: [PATCH 3/3] fix(runtime): restore if-let pattern for request_id guard Co-Authored-By: Claude Sonnet 4.6 --- lib/runtime/src/pipeline/network/ingress/push_handler.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/runtime/src/pipeline/network/ingress/push_handler.rs b/lib/runtime/src/pipeline/network/ingress/push_handler.rs index 146b837a3590..d9df2a23bb55 100644 --- a/lib/runtime/src/pipeline/network/ingress/push_handler.rs +++ b/lib/runtime/src/pipeline/network/ingress/push_handler.rs @@ -132,7 +132,7 @@ impl Drop for RequestMetricsGuard { self.inflight_requests.dec(); self.request_duration .observe(self.start_time.elapsed().as_secs_f64()); - if self.request_id.is_some() { + if let Some(_) = &self.request_id { tracing::info!("request completed"); } } @@ -178,7 +178,7 @@ where m.request_counter.inc(); m.inflight_requests.inc(); m.request_bytes.inc_by(payload.len() as u64); - if request_id.is_some() { + if let Some(_) = &request_id { tracing::info!("request received"); } RequestMetricsGuard {