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

Specify acceptable optimizations for input roots #216

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions build/bazel/remote/execution/v2/remote_execution.proto
Original file line number Diff line number Diff line change
Expand Up @@ -465,6 +465,16 @@ message Action {
// directory, as well as every subdirectory and content blob referred to, MUST
// be in the
// [ContentAddressableStorage][build.bazel.remote.execution.v2.ContentAddressableStorage].
//
// Implementations SHOULD make it appear as if all files in the input root are
// actually present in the specified locations in the input filesystem, and are
// unique copies of the input blobs (unless otherwise specified in the Directory
// tree). However, for optimization reasons implementations MAY break this
// illusion in action-visible ways, e.g. by hardlinking input files to each
// other or to files outside the input root. This will result in certain actions
// not being runnable on those implementations, though for portability actions
// SHOULD avoid being strongly dependent on inode uniqueness, and be tolerant of
// different ways to back a given filesystem path.
Digest input_root_digest = 2;

reserved 3 to 5; // Used for fields moved to [Command][build.bazel.remote.execution.v2.Command].
Expand Down