Title: [267740] trunk/Tools
Revision
267740
Author
aakash_j...@apple.com
Date
2020-09-29 09:27:28 -0700 (Tue, 29 Sep 2020)

Log Message

Limit number of tests to log in case of large number of JSC test failures
https://bugs.webkit.org/show_bug.cgi?id=217085

Reviewed by Jonathan Bedard.

* BuildSlaveSupport/ews-build/steps.py:
(AnalyzeJSCTestsResults.start):

Modified Paths

Diff

Modified: trunk/Tools/BuildSlaveSupport/ews-build/steps.py (267739 => 267740)


--- trunk/Tools/BuildSlaveSupport/ews-build/steps.py	2020-09-29 16:27:13 UTC (rev 267739)
+++ trunk/Tools/BuildSlaveSupport/ews-build/steps.py	2020-09-29 16:27:28 UTC (rev 267740)
@@ -1742,13 +1742,13 @@
         new_stress_failures_to_display = ', '.join(list(new_stress_failures)[:self.NUM_FAILURES_TO_DISPLAY])
         new_binary_failures_to_display = ', '.join(list(new_binary_failures)[:self.NUM_FAILURES_TO_DISPLAY])
 
-        self._addToLog('stderr', '\nFailures in first run: {}'.format(list(first_run_binary_failures) + list(first_run_stress_failures)))
-        self._addToLog('stderr', '\nFailures in second run: {}'.format(list(second_run_binary_failures) + list(second_run_stress_failures)))
+        self._addToLog('stderr', '\nFailures in first run: {}'.format((list(first_run_binary_failures) + list(first_run_stress_failures))[:self.NUM_FAILURES_TO_DISPLAY]))
+        self._addToLog('stderr', '\nFailures in second run: {}'.format((list(second_run_binary_failures) + list(second_run_stress_failures))[:self.NUM_FAILURES_TO_DISPLAY]))
         self._addToLog('stderr', '\nFlaky Tests: {}'.format(flaky_failures_string))
-        self._addToLog('stderr', '\nFailures on clean tree: {}'.format(list(clean_tree_stress_failures) + list(clean_tree_binary_failures)))
+        self._addToLog('stderr', '\nFailures on clean tree: {}'.format(clean_tree_failures_string))
 
         if new_stress_failures or new_binary_failures:
-            self._addToLog('stderr', '\nNew failures: {}\n'.format(list(new_binary_failures) + list(new_stress_failures)))
+            self._addToLog('stderr', '\nNew binary failures: {}.\nNew stress test failures: {}\n'.format(new_binary_failures_to_display, new_stress_failures_to_display))
             self.finished(FAILURE)
             self.build.results = FAILURE
             message = ''

Modified: trunk/Tools/ChangeLog (267739 => 267740)


--- trunk/Tools/ChangeLog	2020-09-29 16:27:13 UTC (rev 267739)
+++ trunk/Tools/ChangeLog	2020-09-29 16:27:28 UTC (rev 267740)
@@ -1,5 +1,15 @@
 2020-09-29  Aakash Jain  <aakash_j...@apple.com>
 
+        Limit number of tests to log in case of large number of JSC test failures
+        https://bugs.webkit.org/show_bug.cgi?id=217085
+
+        Reviewed by Jonathan Bedard.
+
+        * BuildSlaveSupport/ews-build/steps.py:
+        (AnalyzeJSCTestsResults.start):
+
+2020-09-29  Aakash Jain  <aakash_j...@apple.com>
+
         [ews] Refactor NUM_API_FAILURES_TO_DISPLAY
         https://bugs.webkit.org/show_bug.cgi?id=217086
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to