From 2de904f097a8dacba861d3f8c712bc00927108af Mon Sep 17 00:00:00 2001
From: Alejandro Pedraza <alejandro@buoyant.io>
Date: Wed, 13 Dec 2023 15:59:59 -0500
Subject: [PATCH] Improve tracing statements

---
 reinitialize-pods/src/lib.rs | 18 +++++++-----------
 1 file changed, 7 insertions(+), 11 deletions(-)

diff --git a/reinitialize-pods/src/lib.rs b/reinitialize-pods/src/lib.rs
index 75f5bc25..42a3e94b 100644
--- a/reinitialize-pods/src/lib.rs
+++ b/reinitialize-pods/src/lib.rs
@@ -87,16 +87,12 @@ async fn process_events(
                 match tx.try_send(pod.clone()) {
                     Ok(_) => {}
                     Err(TrySendError::Full(_)) => {
-                        tracing::warn!(
-                            name = format!("{namespace}/{name}"),
-                            "Dropped event (channel full)"
-                        );
+                        tracing::warn!(%namespace, %name, "Dropped event (channel full)");
                         metrics.queue_overflow.inc();
                     }
-                    Err(TrySendError::Closed(_)) => tracing::warn!(
-                        name = format!("{namespace}/{name}"),
-                        "Dropped event (channel closed or dropped)"
-                    ),
+                    Err(TrySendError::Closed(_)) => {
+                        tracing::warn!(%namespace, %name, "Dropped event (channel closed or dropped)")
+                    }
                 }
             }
         }
@@ -116,16 +112,16 @@ async fn process_pods(
         let evict_res = pods.evict(&name, &Default::default()).await;
         match evict_res {
             Ok(_) => {
-                tracing::info!(name = format!("{namespace}/{name}"), "Evicting pod");
+                tracing::info!(%namespace, %name, "Evicting pod");
                 metrics.evicted_pods.inc();
                 if let Err(err) =
                     publish_k8s_event(client.clone(), controller_pod_name.clone(), &pod).await
                 {
-                    tracing::warn!(%err, name = format!("{namespace}/{name}"), "Error publishing event");
+                    tracing::warn!(%err, %namespace, %name, "Error publishing event");
                 }
             }
             Err(err) => {
-                tracing::warn!(%err, name = format!("{namespace}/{name}"), "Error evicting pod")
+                tracing::warn!(%err, %namespace, %name, "Error evicting pod")
             }
         }
     }