Title: [127104] trunk/Tools
Revision
127104
Author
[email protected]
Date
2012-08-29 23:19:25 -0700 (Wed, 29 Aug 2012)

Log Message

Random test-webkitpy failures on the buildbot
https://bugs.webkit.org/show_bug.cgi?id=95096

Reviewed by Dirk Pranke.

Suppress occasional errors when running test-webkitpy on GTK builders
by running these tests serially. The 64-bit Release builder is especially
prone to these as it can run up to 24 tests in parallel.

Despite the tests not being run in parallel, the testing only lasts a handful
of seconds more, so this is a worthy trade-off to avoid unnecessary
false-alarm redness on the GTK builders.

* BuildSlaveSupport/build.webkit.org-config/master.cfg:
(RunPythonTests.start):

Modified Paths

Diff

Modified: trunk/Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg (127103 => 127104)


--- trunk/Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg	2012-08-30 05:50:39 UTC (rev 127103)
+++ trunk/Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg	2012-08-30 06:19:25 UTC (rev 127104)
@@ -457,6 +457,14 @@
     command = ["python", "./Tools/Scripts/test-webkitpy"]
     failedTestsFormatString = "%d python tests failed"
 
+    def start(self):
+        platform = self.getProperty('platform')
+        # Python tests are flaky on the GTK builders, running them serially
+        # helps and does not significantly prolong the cycle time.
+        if platform == 'gtk':
+            self.setCommand(self.command + ['--child-processes', '1'])
+        return shell.Test.start(self)
+
     def countFailures(self, cmd):
         logText = cmd.logs['stdio'].getText()
         # We're looking for the line that looks like this: FAILED (failures=2, errors=1)

Modified: trunk/Tools/ChangeLog (127103 => 127104)


--- trunk/Tools/ChangeLog	2012-08-30 05:50:39 UTC (rev 127103)
+++ trunk/Tools/ChangeLog	2012-08-30 06:19:25 UTC (rev 127104)
@@ -1,3 +1,21 @@
+2012-08-29  Zan Dobersek  <[email protected]>
+
+        Random test-webkitpy failures on the buildbot
+        https://bugs.webkit.org/show_bug.cgi?id=95096
+
+        Reviewed by Dirk Pranke.
+
+        Suppress occasional errors when running test-webkitpy on GTK builders
+        by running these tests serially. The 64-bit Release builder is especially
+        prone to these as it can run up to 24 tests in parallel.
+
+        Despite the tests not being run in parallel, the testing only lasts a handful
+        of seconds more, so this is a worthy trade-off to avoid unnecessary
+        false-alarm redness on the GTK builders.
+
+        * BuildSlaveSupport/build.webkit.org-config/master.cfg:
+        (RunPythonTests.start):
+
 2012-08-29  Benjamin Poulain  <[email protected]>
 
         REGRESSION(r126780): Crash using StringImpl::is8Bit before checking if there is an impl
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to