Mark two flaky BES tests with @ignore until a fix is written.

These tests verify corner cases in the bidirectional BES protocol and had
been relying on the in-process gRPC transport with direct execution to
correctly use concurrency primitives. Since the inprocess gRPC transport
was changed in grpc/grpc-java#9319 these tests
are no longer correct and need to be carefully updated to match the new
inprocess gRPC threading model. Updating the tests will take long enough
that temporarily ignoring the tests will meaningfully reduce CI flakes.

PiperOrigin-RevId: 475617300
Change-Id: Ibf27e10b433030ede29fe0587315fbad1a76dbe0

src/test/java/com/google/devtools/build/lib/buildeventservice/BazelBuildEventServiceModuleTest.java

@@ -102,6 +102,7 @@
 import javax.annotation.concurrent.GuardedBy;
 import org.junit.After;
 import org.junit.Before;
+import org.junit.Ignore;
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.rules.TemporaryFolder;
@@ -442,6 +443,9 @@ public void testBeforeSecondCommand_fullyAsync_slowFullCloseIgnored() throws Exc
     events.assertNoWarningsOrErrors();
   }
 
+  // TODO(b/246912214): Deflake this by fixing the threading model to match the upstream gRPC
+  // changes in https://github.com/grpc/grpc-java/pull/9319 that affect InProcessTransport.
+  @Ignore("b/246912214")
   @Test
   public void testBeforeSecondCommand_fullyAsync_slowHalfCloseWarning() throws Exception {
     buildEventService.setDelayBeforeHalfClosingStream(Duration.ofSeconds(10));
@@ -464,6 +468,9 @@ public void testBeforeSecondCommand_fullyAsync_besTimeout_slowFullCloseIgnored()
     events.assertNoWarningsOrErrors();
   }
 
+  // TODO(b/246912214): Deflake this by fixing the threading model to match the upstream gRPC
+  // changes in https://github.com/grpc/grpc-java/pull/9319 that affect InProcessTransport.
+  @Ignore("b/246912214")
   @Test
   public void testBeforeSecondCommand_fullyAsync_besTimeout_slowHalfCloseWarning()
       throws Exception {