commit 469aa7e4e356086d84be4a3344b0b05339c694fd
author John Millikin <jmillikin@stripe.com> Thu Jul 02 01:29:45 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Thu Jul 02 01:31:09 2020 -0700
tree 1961b1596feaeffdd80af8f77e48d742834b62a7
parent 91445cc5147f6e28272d41f96aba14ae4e6139a2

Enable remote options for more commands (any that need to download).

During a migration of an internal project to use the remote downloader, I discovered that `bazel query` doesn't have the remote options registered. This prevents it from being able to download dependencies via the remote downloader.

It seems that the set of commands that can have remote options is a hardcoded list; this PR expands that list to include `sync` and `query`.

Closes #11659.

PiperOrigin-RevId: 319371700

src/main/java/com/google/devtools/build/lib/remote/RemoteModule.java

diff --git a/src/main/java/com/google/devtools/build/lib/remote/RemoteModule.java b/src/main/java/com/google/devtools/build/lib/remote/RemoteModule.java
index e8b329b..6657f52 100644
--- a/src/main/java/com/google/devtools/build/lib/remote/RemoteModule.java
+++ b/src/main/java/com/google/devtools/build/lib/remote/RemoteModule.java
@@ -683,7 +683,7 @@
 
   @Override
   public Iterable<Class<? extends OptionsBase>> getCommandOptions(Command command) {
-    return ImmutableList.of("build", "test", "fetch").contains(command.name())
+    return ImmutableList.of("build", "fetch", "query", "sync", "test").contains(command.name())
         ? ImmutableList.of(RemoteOptions.class, AuthAndTLSOptions.class)
         : ImmutableList.of();
   }