bazel icon indicating copy to clipboard operation
bazel copied to clipboard

Should --sandbox_debug imply --incompatible_sandbox_hermetic_tmp=false?

Open keith opened this issue 1 year ago • 5 comments

Description of the feature request:

I often use --sandbox_debug to grab some paths from the output and inspect the execroot that an action failed with. I've noticed that with --incompatible_sandbox_hermetic_tmp=true (the default), that workflow doesn't work, so I have to flip it to false every time I want to do that. Should that be automatic with --sandbox_debug? Or should my workflow change since maybe we also don't want to keep both codepaths of incompatible_sandbox_hermetic_tmp around forever?

Which category does this issue belong to?

No response

What underlying problem are you trying to solve with this feature?

No response

Which operating system are you running Bazel on?

Linux

What is the output of bazel info release?

7.1.1

If bazel info release returns development version or (@non-git), tell us how you built Bazel.

No response

What's the output of git remote get-url origin; git rev-parse HEAD ?

No response

Have you found anything relevant by searching the web?

No response

Any other information, logs, or outputs that you want to share?

No response

keith avatar May 08 '24 16:05 keith

The paths won't deviate for much longer: https://github.com/bazelbuild/bazel/pull/22001

Also, did you see the command that --sandbox_debug outputs? It drops you into an interactive shell in the sandbox environment in which you can explore the execroot.

fmeum avatar May 08 '24 17:05 fmeum

thanks for the link, so you're saying after that change the output paths will continue to exist? thanks for the command workaround, I'm just so used to the easy copy-paste when i don't need to change the whole context to match bazel

keith avatar May 09 '24 20:05 keith

thanks for the link, so you're saying after that change the output paths will continue to exist?

Yes!

fmeum avatar May 09 '24 22:05 fmeum

awesome, thanks!

keith avatar May 09 '24 22:05 keith

Reopening as this should be closed by #22001 for posterity.

fmeum avatar May 14 '24 13:05 fmeum