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

Path mapping fails for generated multiplex workers #23990

Open
fmeum opened this issue Oct 15, 2024 · 2 comments · May be fixed by #23991
Open

Path mapping fails for generated multiplex workers #23990

fmeum opened this issue Oct 15, 2024 · 2 comments · May be fixed by #23991
Assignees
Labels
P1 I'll work on this now. (Assignee required) team-Local-Exec Issues and PRs for the Execution (Local) team type: bug

Comments

@fmeum
Copy link
Collaborator

fmeum commented Oct 15, 2024

I've been implementing path mapping in few rule sets and I think I'm either implementing something incorrectly or I've found an issue with path mapping + multiplex worker sandboxing. I'm seeing errors where the worker doesn't get run because the executable isn't present in the sandbox. IOException while preparing the execution environment of a worker

Both path mapping and multiplex sandboxing work on their own:

  • Disabling multiplex workers with --noworker_multiplex gets path mapping working.
  • Disabling path mapping with --experimental_output_paths=off gets multiplex sandboxing working.

Here's a minimal repro case as well as the full error (in the README): https://github.com/lucidsoftware/path-mapping-bug-repro You can repro with bazel build example. The worker doesn't do anything, but it doesn't need to in order to repro the bug as we're not getting to the point where the worker actually runs.

Originally posted by @jjudd in #22658 (comment)

@fmeum fmeum self-assigned this Oct 15, 2024
@fmeum fmeum added type: bug team-Local-Exec Issues and PRs for the Execution (Local) team P1 I'll work on this now. (Assignee required) labels Oct 15, 2024
@fmeum fmeum linked a pull request Oct 15, 2024 that will close this issue
@fmeum
Copy link
Collaborator Author

fmeum commented Oct 15, 2024

@bazel-io fork 8.0.0

@fmeum
Copy link
Collaborator Author

fmeum commented Oct 15, 2024

@bazel-io fork 7.4.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 I'll work on this now. (Assignee required) team-Local-Exec Issues and PRs for the Execution (Local) team type: bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant