From b55b0e91be63a961583cfe69da5accbfc7ae8c7a Mon Sep 17 00:00:00 2001
From: Olivier Valentin <ovalenti@redhat.com>
Date: Thu, 2 Jan 2025 16:00:03 +0100
Subject: [PATCH] Turn parse errors into Traces with process state details.

---
 collector/lib/ProcfsScraper.cpp | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/collector/lib/ProcfsScraper.cpp b/collector/lib/ProcfsScraper.cpp
index 22ad007f70..a0097e272b 100644
--- a/collector/lib/ProcfsScraper.cpp
+++ b/collector/lib/ProcfsScraper.cpp
@@ -523,7 +523,10 @@ bool ReadContainerConnections(const char* proc_path, std::shared_ptr<ProcessStor
     uint64_t netns_inode;
     if (!GetNetworkNamespace(dirfd, &netns_inode)) {
       COUNTER_INC(CollectorStats::procfs_could_not_get_network_namespace);
-      CLOG_THROTTLED(ERROR, std::chrono::seconds(10)) << "Could not determine network namespace: " << StrError();
+      CLOG(TRACE) << "Could not determine network namespace: " << StrError();
+      if (process_state) {
+        CLOG(TRACE) << "Process state: " << *process_state;
+      }
       continue;
     }
 
@@ -532,7 +535,10 @@ bool ReadContainerConnections(const char* proc_path, std::shared_ptr<ProcessStor
 
     if (!GetSocketINodes(dirfd, pid, &container_ns_sockets)) {
       COUNTER_INC(CollectorStats::procfs_could_not_get_socket_inodes);
-      CLOG_THROTTLED(ERROR, std::chrono::seconds(10)) << "Could not obtain socket inodes: " << StrError();
+      CLOG(TRACE) << "Could not obtain socket inodes: " << StrError();
+      if (process_state) {
+        CLOG(TRACE) << "Process state: " << *process_state;
+      }
       continue;
     }