kata-containers/cloud-hypervisor
1
0
Fork 0
mirror of https://github.com/cloud-hypervisor/cloud-hypervisor.git synced 2026-08-05 02:19:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

performance-metrics: fix overly broad process cleanup

Browse Source 
Drop the -f flag from the process termination command in
cleanup_stale_processes() so it matches by process name only, not the
full command line. This prevents terminating unrelated processes whose
arguments happen to contain target strings (e.g., the test runner
invoked with --report-file /cloud-hypervisor/report.json).

Use the truncated name 'cloud-hyperviso' because Linux limits process
names to 15 characters.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Anirudh Rayabharam <anrayabh@microsoft.com>
This commit is contained in:
Anirudh Rayabharam
2026-04-16 10:02:00 +00:00
committed by Rob Bradford
parent 005ce38ffd
commit c7a152ee79
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
performance-metrics/src/main.rs
View File

@@ -1297,8 +1297,9 @@ fn run_test_with_timeout(
}
fn cleanup_stale_processes() {
for proc in &["cloud-hypervisor", "iperf3", "ethr"] {
let _ = Command::new("pkill").args(["-9", "-f", proc]).status();
// "cloud-hyperviso" - process name truncated to 15 chars by the kernel
for proc in &["cloud-hyperviso", "iperf3", "ethr"] {
let _ = Command::new("pkill").args(["-9", proc]).status();
}
thread::sleep(Duration::from_secs(2));
}