Merge pull request #2179 from input-output-hk/jpraynaud/2176-retryabl…

…e-devnet-bootstrap-e2e-tests Feat: retryable `devnet` bootstrap errors in e2e tests
input-output-hk · Dec 16, 2024 · dd43599 · dd43599
2 parents 12c09d8 + f7681a3
commit dd43599
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 8 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/mithril-test-lab/mithril-end-to-end/Cargo.toml b/mithril-test-lab/mithril-end-to-end/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "mithril-end-to-end"
-version = "0.4.54"
+version = "0.4.55"
 authors = { workspace = true }
 edition = { workspace = true }
 documentation = { workspace = true }

diff --git a/mithril-test-lab/mithril-end-to-end/src/devnet/runner.rs b/mithril-test-lab/mithril-end-to-end/src/devnet/runner.rs
@@ -119,17 +119,22 @@ impl Devnet {
 
         info!("Bootstrapping the Devnet"; "script" => &bootstrap_script_path.display());
 
-        bootstrap_command
+        let exit_status = bootstrap_command
             .spawn()
             .with_context(|| format!("{bootstrap_script} failed to start"))?
             .wait()
             .await
             .with_context(|| format!("{bootstrap_script} failed to run"))?;
-
-        Ok(Devnet {
-            artifacts_dir: bootstrap_args.artifacts_target_dir.to_owned(),
-            number_of_pool_nodes: bootstrap_args.number_of_pool_nodes,
-        })
+        match exit_status.code() {
+            Some(0) => Ok(Devnet {
+                artifacts_dir: bootstrap_args.artifacts_target_dir.to_owned(),
+                number_of_pool_nodes: bootstrap_args.number_of_pool_nodes,
+            }),
+            Some(code) => Err(anyhow!(RetryableDevnetError(format!(
+                "Bootstrap devnet exited with status code: {code}"
+            )))),
+            None => Err(anyhow!("Bootstrap devnet terminated by signal")),
+        }
     }
 
     /// Factory for test purposes