fix: Check if container run fails when checking version (#729)

Author: gmpinder

process/drivers.rs

@@ -28,7 +28,7 @@ use clap::Args;
 use colored::Colorize;
 use indicatif::{ProgressBar, ProgressStyle};
 use log::{info, trace, warn};
-use miette::{Context, Result};
+use miette::{Context, Result, bail};
 use oci_client::Reference;
 use uuid::Uuid;
 
@@ -306,6 +306,24 @@ fn get_version_run_image(oci_ref: &Reference) -> Result<Version> {
             .build(),
     )?;
 
+    if !output.status.success() {
+        let stderr = String::from_utf8_lossy(&output.stderr);
+        if stderr.contains("docker: failed to register layer: max depth exceeded") {
+            bail!(
+                help = format!(
+                    "Try using podman as your container runtime with `{}`",
+                    "--run-driver=podman".bright_yellow()
+                ),
+                "Docker cannot handle images with too many layers!"
+            );
+        }
+        bail!(
+            "Failed to run version script on the base image {}:\n{}",
+            oci_ref.to_string().cyan(),
+            String::from_utf8_lossy(&output.stderr)
+        );
+    }
+
     if should_remove {
         Driver::remove_image(RemoveImageOpts::builder().image(oci_ref).build())?;
     }

src/bin/bluebuild.rs

@@ -1,11 +1,33 @@
 use blue_build::commands::{BlueBuildArgs, BlueBuildCommand, CommandArgs};
 use blue_build_process_management::{logging::Logger, signal_handler};
 use clap::Parser;
+use colored::Colorize;
 use log::LevelFilter;
 
 fn main() {
     let args = BlueBuildArgs::parse();
 
+    miette::set_hook(Box::new(|_| {
+        Box::new(
+            miette::MietteHandlerOpts::new()
+                .terminal_links(true)
+                .footer(format!(
+                    concat!(
+                        "Try running with `{arg}` for trace logging.\n",
+                        "You can also open an issue at https://github.com/blue-build/cli/issues"
+                    ),
+                    arg = "-vv".bright_yellow(),
+                ))
+                .context_lines(3)
+                .tab_width(4)
+                .wrap_lines(false)
+                .with_cause_chain()
+                .show_related_errors_as_nested()
+                .build(),
+        )
+    }))
+    .expect("Miette hook should be set");
+
     Logger::new()
         .filter_level(args.verbosity.log_level_filter())
         .filter_modules(if args.no_log_filter {