-
-
Notifications
You must be signed in to change notification settings - Fork 15.5k
bazel-remote: init at 2019-01-12 #53840
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
Conversation
@@ -46,6 +46,7 @@ in stdenv.mkDerivation (fBuildAttrs // { | |||
|
|||
# Patching markers to make them deterministic | |||
sed -i 's, -\?[0-9][0-9]*$, 1,' $bazelOut/external/\@*.marker | |||
sed -i '/^ENV:TMP.*/d' $bazelOut/external/\@*.marker |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can change the hash of the dependencies, I've tested pythonPackages.tensorflow
and it still matches, but both bazel-deps
and bazel-watcher
don't build with the version of Bazel we have on master so I can't check.
@GrahamcOfBorg build bazel-remote |
Build fails for linux and darwin (not sure why the status is “neutral”). |
Getting the same issue in #51723 (comment) as well since it uses gazelle too, will continue looking at it. |
84281c8
to
8fe59a5
Compare
Done. Applied the same fix as in #51723 (comment). |
@GrahamcOfBorg build bazel-remote |
Nope, doesn’t build (on Linux):
Also I’m confused about the |
Sorry I updated the ubuntu box so it didn't have sandboxing configured anymore. Could repro the failure with sandboxing enabled, will fix. |
988b83c
to
e8c28d4
Compare
Can't figure out why it's failing. Is there a way to run the sandboxed build in an interactive way? NixOS/nix#903 seems to imply there isn't. |
On linux you could try dropping yourself into a nix shell with the network namespace unshared:
|
e8c28d4
to
28dde48
Compare
28dde48
to
8b272f3
Compare
Won't have time to debug why it's failing on Linux, but I changed the supported platforms to be only Darwin so we can still use it as it is now. |
ping @Profpatsch |
Ah, sorry. Merged. |
Motivation for this change
Adds
bazel-remote
. This the implementation for a bazel cache http backend the official docs refer to, though it's experimental: https://docs.bazel.build/versions/master/remote-caching.html#bazel-remote-cache. It's authored by a member of the Bazel team, even though it's not published under thebazelbuild
org because it's not officially supported. There is no officially supported self-hostable caching backend though.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)