Merge pull request #446 from rawlingsj/runner_log

add extra logging when runner fails to TestUsability
chainguard-dev · May 13, 2023 · 07548a8 · 07548a8
2 parents 88adada + 74da350
commit 07548a8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/pkg/build/build.go b/pkg/build/build.go
@@ -583,7 +583,7 @@ func New(opts ...Option) (*Context, error) {
 
 	// Check that we actually can run things in containers.
 	if !runner.TestUsability() {
-		return nil, fmt.Errorf("unable to run containers")
+		return nil, fmt.Errorf("unable to run containers using %s, specify --runner and one of %s", runner.Name(), GetAllRunners())
 	}
 
 	// Apply build options to the context.

diff --git a/pkg/container/bubblewrap_runner.go b/pkg/container/bubblewrap_runner.go
@@ -85,7 +85,7 @@ func (bw *bubblewrap) Run(cfg *Config, args ...string) error {
 func (bw *bubblewrap) TestUsability() bool {
 	_, err := exec.LookPath("bwrap")
 	if err != nil {
-		bw.logger.Debugf("cannot use bubblewrap for containers: bwrap not found on $PATH")
+		bw.logger.Warnf("cannot use bubblewrap for containers: bwrap not found on $PATH")
 		return false
 	}