Title: [267447] trunk/Tools
Revision
267447
Author
[email protected]
Date
2020-09-22 17:04:52 -0700 (Tue, 22 Sep 2020)

Log Message

Limit number of emails to send for flaky and pre-existing JSC test failures
https://bugs.webkit.org/show_bug.cgi?id=216857

Reviewed by Jonathan Bedard.

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

Modified Paths

Diff

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


--- trunk/Tools/BuildSlaveSupport/ews-build/steps.py	2020-09-22 23:43:06 UTC (rev 267446)
+++ trunk/Tools/BuildSlaveSupport/ews-build/steps.py	2020-09-23 00:04:52 UTC (rev 267447)
@@ -1730,7 +1730,7 @@
 
         flaky_stress_failures = first_run_stress_failures.union(second_run_stress_failures) - first_run_stress_failures.intersection(second_run_stress_failures)
         flaky_binary_failures = first_run_binary_failures.union(second_run_binary_failures) - first_run_binary_failures.intersection(second_run_binary_failures)
-        flaky_failures = list(flaky_binary_failures) + list(flaky_stress_failures)
+        flaky_failures = (list(flaky_binary_failures) + list(flaky_stress_failures))[:self.NUM_FAILURES_TO_DISPLAY]
         flaky_failures_string = ', '.join(flaky_failures)
 
         new_stress_failures = stress_failures_with_patch - clean_tree_stress_failures
@@ -1769,7 +1769,7 @@
             message = ''
             if clean_tree_failures:
                 message = 'Found {} pre-existing JSC test failure{}: {}'.format(len(clean_tree_failures), pluralSuffix, clean_tree_failures_string)
-                for clean_tree_failure in clean_tree_failures:
+                for clean_tree_failure in clean_tree_failures[:self.NUM_FAILURES_TO_DISPLAY]:
                     self.send_email_for_pre_existing_failure(clean_tree_failure)
             if len(clean_tree_failures) > self.NUM_FAILURES_TO_DISPLAY:
                 message += ' ...'

Modified: trunk/Tools/ChangeLog (267446 => 267447)


--- trunk/Tools/ChangeLog	2020-09-22 23:43:06 UTC (rev 267446)
+++ trunk/Tools/ChangeLog	2020-09-23 00:04:52 UTC (rev 267447)
@@ -1,3 +1,13 @@
+2020-09-22  Aakash Jain  <[email protected]>
+
+        Limit number of emails to send for flaky and pre-existing JSC test failures
+        https://bugs.webkit.org/show_bug.cgi?id=216857
+
+        Reviewed by Jonathan Bedard.
+
+        * BuildSlaveSupport/ews-build/steps.py:
+        (AnalyzeJSCTestsResults.start):
+
 2020-09-22  Sihui Liu  <[email protected]>
 
         REGRESSION(r266075): WebContent process crashes at TextManipulationController::getPath
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to