do not run ledger-api-integration-tests on macOS (digital-asset#1318)

* do not run ledger-api-integration-tests on macOS * Update build.sh Co-Authored-By: Moritz Kiefer <moritz.kiefer@purelyfunctional.org>
kgeonw · May 23, 2019 · 4922cdf · 4922cdf
1 parent 5cddf53
commit 4922cdf
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/build.sh b/build.sh
@@ -24,7 +24,11 @@ ARTIFACT_DIRS="${BUILD_ARTIFACTSTAGINGDIRECTORY:-$PWD}"
   # https://github.com/bazelbuild/bazel/issues/6394#issuecomment-436234594.
   bazel build -j 200 //... --experimental_execution_log_file "$ARTIFACT_DIRS/build_execution${execution_log_postfix}.log"
 )
-bazel test -j 200 //... --experimental_execution_log_file "$ARTIFACT_DIRS/test_execution${execution_log_postfix}.log"
+tag_filter=""
+if [[ "$execution_log_postfix" == "_Darwin" ]]; then
+  tag_filter="-dont-run-on-darwin"
+fi
+bazel test -j 200 //... --test_tag_filters "$tag_filter" --experimental_execution_log_file "$ARTIFACT_DIRS/test_execution${execution_log_postfix}.log"
 # Make sure that Bazel query works.
 bazel query 'deps(//...)' > /dev/null
 # Check that we can load damlc in ghci

diff --git a/ledger/ledger-api-integration-tests/BUILD.bazel b/ledger/ledger-api-integration-tests/BUILD.bazel
@@ -69,8 +69,12 @@ da_scala_test_suite(
     ],
     # this test suite is very intensive, long, and relying on timeouts. Moreover,
     # the test suite already parallelizes each scala test, so we'll still be busy
-    # only with this one.
-    tags = ["exclusive"],
+    # only with this one. for the same reason we do not run it on macos -- it takes
+    # up to 20 minutes on our macOS ci workers.
+    tags = [
+        "dont-run-on-darwin",
+        "exclusive",
+    ],
     deps = [
         ":ledger-api-integration-tests-lib",
     ] + dependencies,