sandbox: Better error messages and the noisy debug logs of the namespace-runner now have to be explicitly activated via --sandbox_debug.

Fixes #424.

--
MOS_MIGRATED_REVID=102566625

1 files changed, 8 insertions, 1 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/sandbox/SandboxActionContextProvider.java b/src/main/java/com/google/devtools/build/lib/sandbox/SandboxActionContextProvider.java
index 20b64c87ca..61cd2cdbd1 100644
--- a/src/main/java/com/google/devtools/build/lib/sandbox/SandboxActionContextProvider.java
+++ b/src/main/java/com/google/devtools/build/lib/sandbox/SandboxActionContextProvider.java
@@ -35,10 +35,17 @@ public class SandboxActionContextProvider extends ActionContextProvider {
   public SandboxActionContextProvider(
       BlazeRuntime runtime, BuildRequest buildRequest, ExecutorService backgroundWorkers) {
     boolean verboseFailures = buildRequest.getOptions(ExecutionOptions.class).verboseFailures;
+    boolean sandboxDebug = buildRequest.getOptions(SandboxOptions.class).sandboxDebug;
     Builder<ActionContext> strategies = ImmutableList.builder();
 
     if (OS.getCurrent() == OS.LINUX) {
-      strategies.add(new LinuxSandboxedStrategy(runtime, verboseFailures, backgroundWorkers));
+      strategies.add(
+          new LinuxSandboxedStrategy(
+              runtime.getClientEnv(),
+              runtime.getDirectories(),
+              backgroundWorkers,
+              verboseFailures,
+              sandboxDebug));
     }
 
     this.strategies = strategies.build();