From 60de78519c873597b6740fcd3136d11db58173c6 Mon Sep 17 00:00:00 2001
From: Jay Conrod <jay@engflow.com>
Date: Wed, 27 Dec 2023 10:50:22 -0800
Subject: [PATCH] .bazelrc: fix redundancy that caused a Bazel crash (#275)

Not really sure why, but specifying JWT authentication flags multiple times
causes Bazel to crash with no output, exit code 37.

Remove the redundant --config=opal_auth from the opal_bes and magnesite_bes
configurations. Add to test-all.py when remote execution is disabled.

I think we don't see this in CI because credential helper flags are safe
to specify more than once but --remote_header is not.

For linear/PE-2268
---
 .bazelrc          | 2 --
 infra/test-all.py | 1 +
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/.bazelrc b/.bazelrc
index 56175d29..a4d04210 100644
--- a/.bazelrc
+++ b/.bazelrc
@@ -69,7 +69,6 @@ build:opal --config=opal_bes
 build:opal --config=opal_auth
 build:opal --remote_executor=grpcs://opal.cluster.engflow.com
 
-build:opal_bes --config=opal_auth
 build:opal_bes --bes_backend=grpcs://opal.cluster.engflow.com
 build:opal_bes --bes_lifecycle_events
 build:opal_bes --bes_results_url=https://opal.cluster.engflow.com/invocation/
@@ -79,7 +78,6 @@ build:magnesite --config=magnesite_bes
 build:magnesite --config=magnesite_auth
 build:magnesite --remote_executor=grpcs://magnesite.cluster.engflow.com
 
-build:magnesite_bes --config=magnesite_auth
 build:magnesite_bes --bes_backend=grpcs://magnesite.cluster.engflow.com
 build:magnesite_bes --bes_lifecycle_events
 build:magnesite_bes --bes_results_url=https://magnesite.cluster.engflow.com/invocation/
diff --git a/infra/test-all.py b/infra/test-all.py
index 29d3a1fa..d91171d6 100644
--- a/infra/test-all.py
+++ b/infra/test-all.py
@@ -21,6 +21,7 @@ def main():
         flags += [
             "--platforms=//platform/" + os_arch,
             "--config=opal_bes",
+            "--config=opal_auth",
         ]
 
     targets = [