Title: [157284] trunk/Tools
Revision
157284
Author
k...@webkit.org
Date
2013-10-10 18:47:38 -0700 (Thu, 10 Oct 2013)

Log Message

webkit-patch failure-reason has a quite low limit for 'too many failures'
https://bugs.webkit.org/show_bug.cgi?id=122556

Patch by Gustavo Noronha Silva <gustavo.noro...@collabora.com> on 2013-10-10
Reviewed by Darin Adler.

* Scripts/webkitpy/tool/commands/queries.py:
(FailureReason._explain_failures_for_builder): adjust the number of too many failures
to match what the layout test runners use these days.

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (157283 => 157284)


--- trunk/Tools/ChangeLog	2013-10-11 01:41:52 UTC (rev 157283)
+++ trunk/Tools/ChangeLog	2013-10-11 01:47:38 UTC (rev 157284)
@@ -1,5 +1,16 @@
 2013-10-10  Gustavo Noronha Silva  <gustavo.noro...@collabora.com>
 
+        webkit-patch failure-reason has a quite low limit for 'too many failures'
+        https://bugs.webkit.org/show_bug.cgi?id=122556
+
+        Reviewed by Darin Adler.
+
+        * Scripts/webkitpy/tool/commands/queries.py:
+        (FailureReason._explain_failures_for_builder): adjust the number of too many failures
+        to match what the layout test runners use these days.
+
+2013-10-10  Gustavo Noronha Silva  <gustavo.noro...@collabora.com>
+
         webkit-patch failure-reason should allow specifying failures one is interested in
         https://bugs.webkit.org/show_bug.cgi?id=122554
 

Modified: trunk/Tools/Scripts/webkitpy/tool/commands/queries.py (157283 => 157284)


--- trunk/Tools/Scripts/webkitpy/tool/commands/queries.py	2013-10-11 01:41:52 UTC (rev 157283)
+++ trunk/Tools/Scripts/webkitpy/tool/commands/queries.py	2013-10-11 01:47:38 UTC (rev 157284)
@@ -290,9 +290,9 @@
                 print "No results build %s (r%s)" % (build._number, build.revision())
                 continue
             failures = set(latest_results.failing_tests())
-            if len(failures) >= 20:
+            if len(failures) >= 500:
                 # FIXME: We may need to move this logic into the LayoutTestResults class.
-                # The buildbot stops runs after 20 failures so we don't have full results to work with here.
+                # The buildbot stops runs after 500 failures so we don't have full results to work with here.
                 print "Too many failures in build %s (r%s), ignoring." % (build._number, build.revision())
                 continue
             fixed_results = results_to_explain - failures
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to