feat: improve BinaryNotFound error with a hint

Signed-off-by: Malo Polese <[email protected]>
rik-org · May 11, 2023 · 581c727 · 581c727
1 parent 115126a
commit 581c727
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/firepilot/src/builder/executor.rs b/firepilot/src/builder/executor.rs
@@ -80,7 +80,7 @@ impl FirecrackerExecutorBuilder {
             .or_else(Self::find_binary_from_path)
             .or_else(Self::find_binary_from_current_directory)
             .map(|p| Ok(p))
-            .unwrap_or(Err(BuilderError::BinaryNotFound))
+            .unwrap_or(Err(BuilderError::BinaryNotFound("Check if FIRECRACKER_LOCATION environment variable is correctly set. For more information check https://docs.rs/firepilot/ ".to_string())))
     }
 
     /// Create a new firecracker executor, it will try to determine the binary location, but you can

diff --git a/firepilot/src/builder/mod.rs b/firepilot/src/builder/mod.rs
@@ -70,7 +70,7 @@ pub enum BuilderError {
     /// The field is required but was not provided in the builder object
     MissingRequiredField(String),
     /// Happens when using auto methods to detect firecracker /jailer binary
-    BinaryNotFound,
+    BinaryNotFound(String),
 }
 
 /// Generic trait which all builder componenet must implement in order to be