gnosis/lighthouse
1
0
Fork 0
mirror of https://github.com/sigp/lighthouse.git synced 2026-05-07 08:52:54 +00:00
Code Issues Packages Projects Releases Wiki Activity

Correctly compute process times during ProcessHealth::observe (#8793)

Browse Source 
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 <mjladson@pm.me>
This commit is contained in:
Mac L
2026-02-14 12:26:25 +04:00
committed by GitHub
parent 68ad9758a3
commit a3a74d8988
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
common/health_metrics/src/observe.rs
View File

@@ -121,7 +121,7 @@ impl Observe for ProcessHealth {
pid_mem_shared_memory_size: process_mem.shared(), pid_mem_shared_memory_size: process_mem.shared(),
pid_process_seconds_total: process_times.busy().as_secs() pid_process_seconds_total: process_times.busy().as_secs()
+ process_times.children_system().as_secs() + process_times.children_system().as_secs()
+ process_times.children_system().as_secs(), + process_times.children_user().as_secs(),
}) })
} }
} }