commit 02a6e0f8ac34a4ca44cad6c8b2eba81645da10e4
author ruperts <ruperts@google.com> Tue Aug 07 08:08:09 2018 -0700
committer Copybara-Service <copybara-piper@google.com> Tue Aug 07 08:09:44 2018 -0700
tree 11f603225d0a6403285bf2b836057384d6f35df1
parent 4d8f31f47a92144dad620442bfedf4e772758b7e

Temporarily disable the CommandUsingProcessWrapperTest system time upper bound check as it is intermittently failing under very heavy load.

RELNOTES: None.
PiperOrigin-RevId: 207716645

src/test/java/com/google/devtools/build/lib/shell/ExecutionStatisticsTestUtil.java

diff --git a/src/test/java/com/google/devtools/build/lib/shell/ExecutionStatisticsTestUtil.java b/src/test/java/com/google/devtools/build/lib/shell/ExecutionStatisticsTestUtil.java
index ff834491..7030e19 100644
--- a/src/test/java/com/google/devtools/build/lib/shell/ExecutionStatisticsTestUtil.java
+++ b/src/test/java/com/google/devtools/build/lib/shell/ExecutionStatisticsTestUtil.java
@@ -46,7 +46,9 @@
     Duration userTimeLowerBound = userTimeToSpend;
     Duration userTimeUpperBound = userTimeToSpend.plusSeconds(9);
     Duration systemTimeLowerBound = systemTimeToSpend;
-    Duration systemTimeUpperBound = systemTimeToSpend.plusSeconds(9);
+
+    // TODO(b/110456205) This check fails under very heavy load, investigate why and re-enable it
+    // Duration systemTimeUpperBound = systemTimeToSpend.plusSeconds(9);
 
     Command command = new Command(fullCommandLine.toArray(new String[0]));
     CommandResult commandResult = command.execute();
@@ -62,6 +64,8 @@
 
     Duration systemTime = resourceUsage.get().getSystemExecutionTime();
     assertThat(systemTime).isAtLeast(systemTimeLowerBound);
-    assertThat(systemTime).isAtMost(systemTimeUpperBound);
+
+    // TODO(b/110456205) This check fails under very heavy load, investigate why and re-enable it
+    // assertThat(systemTime).isAtMost(systemTimeUpperBound);
   }
 }