[Webkit-unassigned] [Bug 222601] [JSC] detect infrastructure failure for remote stress tests

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 13 08:39:06 PDT 2021


https://bugs.webkit.org/show_bug.cgi?id=222601

--- Comment #16 from Angelos Oikonomopoulos <angelos at igalia.com> ---
(In reply to Mark Lam from comment #15)
> (In reply to Angelos Oikonomopoulos from comment #14)
> > (In reply to Yusuke Suzuki from comment #11)
[...]
> > The above messages don't come from any script generated by it AFAIK. (Does
> > @Mark refer to Mark Lam? If so, please let me know and I can ping them
> > directly.)
> 
> Yes, Yusuke was referring to me.  These messages came from internal tests. 
> Before your patch, running the script did not produce this output.  After
> your patch, it does.  I suspect that the tests were always printing these
> output.  However, your change somehow expose them.  Do you have any idea
> where your patch enables more verbose stderr/stdout output to be shown?

I've taken a second look and I don't see any instances where the patch changes where stdout/stderr go, but it's tricky code. I can't see any extra output in my testing of course (I just checked again both with local and --remote runs).

webkitruby/jsc-stress-test-writer-default.rb defines a number of handlers that the patch modifies. Are you at liberty to share what test kind is producing the extraneous output? I.e. which run* function is invoked in run-jsc-stress-tests? If so, then I could look at a diff of the generated code in detail.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20210413/a087c248/attachment.htm>


More information about the webkit-unassigned mailing list