Title: [289811] trunk/Tools
Revision
289811
Author
[email protected]
Date
2022-02-15 07:10:58 -0800 (Tue, 15 Feb 2022)

Log Message

[JSC] Speed up getStatusMap and increase robustness
https://bugs.webkit.org/show_bug.cgi?id=236559

Reviewed by Adrian Perez de Castro.

Collect test results in parallel. Also, ignore ssh errors during
collection (e.g. because the remote host is down) -- the main retry
loop will take care of that for us.

This should both avoid needless failures like
https://build.webkit.org/#/builders/31/builds/2432 and reduce the
latency when there are multiple remotes (e.g. for MIPS).

* Scripts/run-jsc-stress-tests:

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (289810 => 289811)


--- trunk/Tools/ChangeLog	2022-02-15 14:53:00 UTC (rev 289810)
+++ trunk/Tools/ChangeLog	2022-02-15 15:10:58 UTC (rev 289811)
@@ -1,3 +1,20 @@
+2022-02-15  Angelos Oikonomopoulos  <[email protected]>
+
+        [JSC] Speed up getStatusMap and increase robustness
+        https://bugs.webkit.org/show_bug.cgi?id=236559
+
+        Reviewed by Adrian Perez de Castro.
+
+        Collect test results in parallel. Also, ignore ssh errors during
+        collection (e.g. because the remote host is down) -- the main retry
+        loop will take care of that for us.
+
+        This should both avoid needless failures like
+        https://build.webkit.org/#/builders/31/builds/2432 and reduce the
+        latency when there are multiple remotes (e.g. for MIPS).
+
+        * Scripts/run-jsc-stress-tests:
+
 2022-02-14  Kimmo Kinnunen  <[email protected]>
 
         ExtensionsGL is not useful class, it should be removed

Modified: trunk/Tools/Scripts/run-jsc-stress-tests (289810 => 289811)


--- trunk/Tools/Scripts/run-jsc-stress-tests	2022-02-15 14:53:00 UTC (rev 289810)
+++ trunk/Tools/Scripts/run-jsc-stress-tests	2022-02-15 15:10:58 UTC (rev 289811)
@@ -2769,8 +2769,7 @@
         # well be test results on a remoteHost that got rebooted
         # (note, the test results are tagged with a run ID, so we'll
         # ignore any stale results from a previous run).
-        $remoteHosts.each_with_index {
-            | host, remoteIndex |
+        forEachRemote($remoteHosts, :dropOnFailure => true, :timeout => REMOTE_TIMEOUT) { |_, host|
             runnerDir = "#{host.remoteDirectory}/#{$outputDir.basename}/.runner"
             output = sshRead("if test -d #{runnerDir}; then cd #{runnerDir}; else false; fi && " + find_cmd, host, :ignoreFailure => true)
             output.split(/\n/).each {
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to