Skip to content

Commit 1b479e4

Browse files
russell-islamrbradford
authored andcommitted
performance-metrics: Remove duplicate remote_command
Remove the local remote_command() function from performance_tests.rs. The identical function is now available from test_infra via the existing glob import. Signed-off-by: Muminul Islam <muislam@microsoft.com>
1 parent c8bfac6 commit 1b479e4

1 file changed

Lines changed: 0 additions & 20 deletions

File tree

performance-metrics/src/performance_tests.rs

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -124,26 +124,6 @@ fn performance_test_new_guest(disk_config: Box<dyn DiskConfig>) -> Guest {
124124
Guest::new_from_ip_range(disk_config, "172.19", 0)
125125
}
126126

127-
fn remote_command(api_socket: &str, command: &str, arg: Option<&str>) -> bool {
128-
let mut cmd = std::process::Command::new(clh_command("ch-remote"));
129-
cmd.args([&format!("--api-socket={api_socket}"), command]);
130-
131-
if let Some(arg) = arg {
132-
cmd.arg(arg);
133-
}
134-
let output = cmd.output().unwrap();
135-
if output.status.success() {
136-
true
137-
} else {
138-
eprintln!(
139-
"Error running ch-remote command: {:?}\nstderr: {}",
140-
&cmd,
141-
String::from_utf8_lossy(&output.stderr)
142-
);
143-
false
144-
}
145-
}
146-
147127
pub fn performance_net_throughput(control: &PerformanceTestControl) -> f64 {
148128
let test_timeout = control.test_timeout;
149129
let (rx, bandwidth) = control.net_control.unwrap();

0 commit comments

Comments
 (0)