Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remote falls back to local, not sandboxed #21725

Closed
Tracked by #19904
jmmv opened this issue Mar 18, 2024 · 3 comments
Closed
Tracked by #19904

Remote falls back to local, not sandboxed #21725

jmmv opened this issue Mar 18, 2024 · 3 comments
Labels
P2 We'll consider working on this in future. (Assignee optional) team-Local-Exec Issues and PRs for the Execution (Local) team type: bug

Comments

@jmmv
Copy link
Contributor

jmmv commented Mar 18, 2024

Description of the bug:

We have a few actions that require either remote execution or sandboxed execution: they cannot be built with the local strategy.

We configure Bazel like this:

build --internal_spawn_scheduler
build --spawn_strategy=dynamic
build --remote_local_fallback
build --strategy=BootstrapGNUMake=remote,sandboxed
... some more strategy flags exactly like the previous one ...

But I've noticed that, when a remote action fails due to networking issues, Bazel reruns it without the sandbox and thus the action fails with the local strategy.

I found the --remote_local_fallback_strategy flag, which I thought would resolve our problems... but then this flag is deprecated and its docstring claims that it's a no-op. Then I went to #7480 and, from my understanding of that bug, our strategy flags are already correctly configured. --spawn_strategy says dynamic, and the dynamic scheduler defaults to worker,sandboxed for the local path.

Is there a bug, or is something misconfigured?

Which category does this issue belong to?

Local Execution

What's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.

No response

Which operating system are you running Bazel on?

Linux but this is irrelevant

What is the output of bazel info release?

release 6.5.0

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

Is this a regression? If yes, please try to identify the Bazel commit where the bug was introduced.

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

@github-actions github-actions bot added the team-Local-Exec Issues and PRs for the Execution (Local) team label Mar 18, 2024
@jin jin added P2 We'll consider working on this in future. (Assignee optional) and removed untriaged labels Mar 19, 2024
@tjgq tjgq mentioned this issue Mar 19, 2024
12 tasks
@jmmv
Copy link
Contributor Author

jmmv commented Mar 19, 2024

@tjgq I see you referenced this issue in #19904 but I'm not sure what that means for what I reported here. Is it possible to select "sandboxed" for the local fallback strategy? Or is this currently not doable given that the flag says it's a no-op?

@ulrfa
Copy link
Contributor

ulrfa commented Mar 20, 2024

Is this a duplicate of #15519?

@jmmv
Copy link
Contributor Author

jmmv commented Mar 20, 2024

Yeah, it looks like it.

@jmmv jmmv closed this as completed Mar 20, 2024
@brentleyjones brentleyjones closed this as not planned Won't fix, can't repro, duplicate, stale Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 We'll consider working on this in future. (Assignee optional) team-Local-Exec Issues and PRs for the Execution (Local) team type: bug
Projects
None yet
Development

No branches or pull requests

6 participants