From a3a74d89881265ebe44299dbe7df7331561e012d Mon Sep 17 00:00:00 2001 From: Mac L Date: Sat, 14 Feb 2026 12:26:25 +0400 Subject: [PATCH] Correctly compute process times during `ProcessHealth::observe` (#8793) I believe I found a bug where during computation of `pid_process_seconds_total` we add `children_system` twice. I assume that it was originally intended to add `children_system` and `children_user` once each Co-Authored-By: Mac L --- common/health_metrics/src/observe.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/common/health_metrics/src/observe.rs b/common/health_metrics/src/observe.rs index 81bb8e6f7e..5bc3770301 100644 --- a/common/health_metrics/src/observe.rs +++ b/common/health_metrics/src/observe.rs @@ -121,7 +121,7 @@ impl Observe for ProcessHealth { pid_mem_shared_memory_size: process_mem.shared(), pid_process_seconds_total: process_times.busy().as_secs() + process_times.children_system().as_secs() - + process_times.children_system().as_secs(), + + process_times.children_user().as_secs(), }) } }