Skip to content

Commit

Permalink
chore: replace eprintln calls with logging
Browse files Browse the repository at this point in the history
  • Loading branch information
j-lanson authored and alilleybrinker committed Sep 13, 2024
1 parent f447b7c commit 24236e0
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions hipcheck/src/plugin/manager.rs
Original file line number Diff line number Diff line change
Expand Up @@ -75,15 +75,15 @@ impl PluginExecutor {
// on the cmdline is not already in use, but it is still possible for that
// port to become unavailable between our check and the plugin's bind attempt.
// Hence the need for subsequent attempts if we get unlucky
eprintln!("Starting plugin '{}'", plugin.name);
log::debug!("Starting plugin '{}'", plugin.name);
let mut spawn_attempts: usize = 0;
while spawn_attempts < self.max_spawn_attempts {
// Find free port for process. Don't retry if we fail since this means all
// ports in the desired range are already bound
let port = self.get_available_port()?;
let port_str = port.to_string();
// Spawn plugin process
eprintln!("Spawning '{}' on port {}", &plugin.entrypoint, port_str);
log::debug!("Spawning '{}' on port {}", &plugin.entrypoint, port_str);
let Ok(mut proc) = Command::new(&plugin.entrypoint)
.args(["--port", port_str.as_str()])
// @Temporary - directly forward stdout/stderr from plugin to shell
Expand Down

0 comments on commit 24236e0

Please sign in to comment.