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.

Sign up for GitHub

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

Jump to bottom

Create output directories for remote execution #15477

Closed
wants to merge 1 commit into from
Closed

Create output directories for remote execution #15477

wants to merge 1 commit into from

Commits on May 12, 2022

  1. Create output directories for remote execution 

    By explicitly declaring output directories as inputs to a remote action,
this commit ensures that these directories are created by the remote
executor prior to the execution of the action. This brings the behavior
of remote execution regarding tree artifacts in line with that of local
or sandboxed execution.

Getting the tests to pass requires modifying a check in Bazel's own
remote worker implementation: Previously, the worker explicitly verified
that output directories don't exist after the inputs have been staged.
This behavior is not backed by the spec and has thus been modified: Now,
it is only checked that the output directories either don't exist or are
directories.

Fixes #6393

Closes #15366.

PiperOrigin-RevId: 447451303
    @fmeum @ckolli5
    fmeum authored and ckolli5 committed May 12, 2022
    Configuration menu
    Copy the full SHA
    a20cda1 View commit details
    Browse the repository at this point in the history