-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
feature request: A mode for --experimental_remote_download_outputs to download testlogs only #8934
Comments
have you tried |
@buchgr No Can you reopen the issue? |
|
@buchgr Yes I did. Bazel downloads the binaries and it bursted CI. I had to add https://github.com/envoyproxy/envoy/blob/50fb561c23078e01efd4dc3852812efbf5ca8936/.bazelrc#L107. I just tried now again and it downloaded binary. The command I ran:
at the commit above. |
@lizan ahh I see what you mean. Thanks for the reproducer. Please note that with UPDATE: I am looking into a fix. |
Yeah I'd like to setup CI Test Summary like this without download test binaries. So it will be nice to have all |
This introduces two new behaviours: 1) bazel test //:foo_test downloads bazel-testlogs/foo_test/test.{log|xml} 2) bazel build //:foo_test downloads bazel-bin/foo_test Fixes bazelbuild#8934
I have sent out #8947 to address this issue. Maybe you could try it out on envoy? |
I tested it with the command line you provided and things work fine! |
Thanks @buchgr I was able to verify in Envoy. |
Description of the problem / feature request:
Feature requests: what underlying problem are you trying to solve with this feature?
While setting up Bazel RBE with Envoy CI, we want the CI download less artifacts from RBE to save network bandwidth and disk, while we want to be able to inspect testlogs individually.
Currently the setup is --test_output=all and --experimental_remote_download_outputs=minimal so all test outputs are printed to console, the test log are already downloaded but not write to the disk, it would be nice bazel-testlogs is set up, and better with test.xml, so CI system can interpret them.
What operating system are you running Bazel on?
Ubuntu 16.04
What's the output of
bazel info release
?release 0.28.0
The text was updated successfully, but these errors were encountered: