aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/main/java/com/google/devtools/build/lib/sandbox
diff options
context:
space:
mode:
authorGravatar Siddhartha Bagaria <sbagaria@grailbio.com>2018-05-17 05:09:52 -0700
committerGravatar Copybara-Service <copybara-piper@google.com>2018-05-17 05:11:03 -0700
commita61b53a8ef65881c6c4a69f16edc52ed7f1662a3 (patch)
tree9e6178233c5441cc3d3159fba103f912b2470060 /src/main/java/com/google/devtools/build/lib/sandbox
parente1ada6d351cb714c9e5694e1e58cd2b761c84784 (diff)
uid related fixes in docker sandbox
uid and gid of -1 (set when not linux) can not really be used anywhere. Closes #5165. PiperOrigin-RevId: 196974660
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/sandbox')
-rw-r--r--src/main/java/com/google/devtools/build/lib/sandbox/DockerSandboxedSpawnRunner.java6
1 files changed, 6 insertions, 0 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/sandbox/DockerSandboxedSpawnRunner.java b/src/main/java/com/google/devtools/build/lib/sandbox/DockerSandboxedSpawnRunner.java
index cfbfdc4859..67c495248a 100644
--- a/src/main/java/com/google/devtools/build/lib/sandbox/DockerSandboxedSpawnRunner.java
+++ b/src/main/java/com/google/devtools/build/lib/sandbox/DockerSandboxedSpawnRunner.java
@@ -292,6 +292,12 @@ final class DockerSandboxedSpawnRunner extends AbstractSandboxSpawnRunner {
return baseImage;
}
+ // We only need to create a customized image, if we're running on Linux, as Docker on macOS
+ // and Windows doesn't map users from the host into the container anyway.
+ if (OS.getCurrent() != OS.LINUX) {
+ return baseImage;
+ }
+
return imageMap.computeIfAbsent(
baseImage,
(image) -> {