Bug 201426 – Allow run-jsc-stress-tests still run if some of the remote hosts are not available

• Status: REOPENED
• Component: JavaScriptCore

[Caio Lima]

Now, if one of the listed remote hosts is not available, we will abort the run of the script. This is a problem mainly to ARMv7 and MIPS EWS, because we run tests in multiple devices, and if any of the bots are unavailable, we don't process any patch in the queue. Keeping this script executing if we have at least 1 remote host available will increase the robustness of those EWS bots.

[Caio Lima]

Created attachment 377901 [details]
WIP - Test

[Caio Lima]

Created attachment 377914 [details]
Patch

[Guillaume Emont]

Comment on attachment 377914 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=377914&action=review

The change looks good to me. It gives me loads of ideas of things to clean up in that script (such as having a function with fewer parameters to replace all the mysys("ssh", ....) calls), but that's beside the point.

> Tools/ChangeLog:3
> +        Allow run-jsc-stress-tests still run if some of the remote hosts are not available

grammar/clarity nitpick: Allow run-jsc-stress-tests to keep on running if some of the remote hosts are not available

[Paulo Matos]

Took a look at it as well, and besides Guillaume's nit, I have nothing to add. This is a very useful patch.

[Caio Lima]

Created attachment 400808 [details]
Patch

[Caio Lima]

Ping Review.

We are particularly interested into this patch because of our constant issues into MIPS EWS queue. The current hardware we have there is very faulty and we are using a dozen of them to run tests faster. Being able to keep consuming queue items if 1 or 2 boards are temporarily unavailable would be great to avoid skipping a lot of important patches. We already started looking for alternative MIPS hardware, but in the meantime, this patch can benefit a lot MIPS EWS queue wait time.

[Saam Barati]

Comment on attachment 400808 [details]
Patch

r=me

[Caio Lima]

Comment on attachment 400808 [details]
Patch

Thank you very much for the review!

[EWS]

Committed r262696: <https://trac.webkit.org/changeset/262696>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 400808 [details].

[Radar WebKit Bug Importer]

<rdar://problem/64089436>

[Yusuke Suzuki]

Re-opened since this is blocked by bug 213097