[PATCH] D56064: More tolerance for flaky tests in libc++ on NetBSD

Phabricator via Phabricator reviews at reviews.llvm.org
Sun Dec 30 15:08:44 PST 2018


This revision was automatically updated to reflect the committed changes.
Closed by commit rCXX350170: More tolerance for flaky tests in libc++ on NetBSD (authored by kamil, committed by ).
Herald added a subscriber: ldionne.

Repository:
  rCXX libc++

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56064/new/

https://reviews.llvm.org/D56064

Files:
  utils/libcxx/test/format.py


Index: utils/libcxx/test/format.py
===================================================================
--- utils/libcxx/test/format.py
+++ utils/libcxx/test/format.py
@@ -12,6 +12,7 @@
 import os
 import time
 import random
+import platform
 
 import lit.Test        # pylint: disable=import-error
 import lit.TestRunner  # pylint: disable=import-error
@@ -202,6 +203,12 @@
                           for f in os.listdir(local_cwd) if f.endswith('.dat')]
             is_flaky = self._get_parser('FLAKY_TEST.', parsers).getValue()
             max_retry = 3 if is_flaky else 1
+
+            # LIBC++ tests tend to be more flaky on NetBSD, so add more retries.
+            # We don't do this on other platforms because it's slower.
+            if platform.system() in ['NetBSD']:
+                max_retry = max_retry * 3
+
             for retry_count in range(max_retry):
                 cmd, out, err, rc = self.executor.run(exec_path, [exec_path],
                                                       local_cwd, data_files,


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D56064.179743.patch
Type: text/x-patch
Size: 1034 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20181230/72f5722b/attachment.bin>


More information about the libcxx-commits mailing list