Better logging on agent

nymtech · Oct 30, 2024 · acb7be7 · acb7be7
1 parent ab03207
commit acb7be7
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 2 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/nym-node-status-agent/Cargo.toml b/nym-node-status-agent/Cargo.toml
@@ -4,7 +4,7 @@
 
 [package]
 name = "nym-node-status-agent"
-version = "0.1.0"
+version = "0.1.2"
 authors.workspace = true
 repository.workspace = true
 homepage.workspace = true

diff --git a/nym-node-status-agent/src/cli.rs b/nym-node-status-agent/src/cli.rs
@@ -97,6 +97,7 @@ async fn submit_results(
 ) -> anyhow::Result<()> {
     let target_url = format!("{}/{}/{}", server_addr, URL_BASE, testrun_id);
     let client = reqwest::Client::new();
+
     let res = client
         .post(target_url)
         .body(probe_outcome)

diff --git a/nym-node-status-agent/src/probe.rs b/nym-node-status-agent/src/probe.rs
@@ -40,6 +40,12 @@ impl GwProbe {
         match command.spawn() {
             Ok(child) => {
                 if let Ok(output) = child.wait_with_output() {
+                    if !output.status.success() {
+                        let out = String::from_utf8_lossy(&output.stdout);
+                        let err = String::from_utf8_lossy(&output.stderr);
+                        tracing::error!("Probe exited with {:?}:\n{}\n{}", output.status, out, err);
+                    }
+
                     return String::from_utf8(output.stdout)
                         .unwrap_or("Unable to get log from test run".to_string());
                 }