Summary: | Add some debug output for run-jsc-stress-tests | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Zhifei Fang <zhifei_fang> | ||||||||
Component: | New Bugs | Assignee: | Zhifei Fang <zhifei_fang> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | aakash_jain, jbedard, saam, tsavell, webkit-bug-importer, zhifei_fang | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Zhifei Fang
2020-04-24 15:28:44 PDT
Created attachment 397518 [details]
Patch
I am using this to trigger a EWS to see where we hit for the JSC time out issue Most the timeout happened at the end of the running, it could be timeout in the detect failure stage, add these output to help me locate where are the timeout Comment on attachment 397518 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=397518&action=review > Tools/ChangeLog:3 > + Add some debug output for run-jsc-stress-tests Can we embed some of the why in the commit message? Created attachment 397522 [details]
Patch
(In reply to Jonathan Bedard from comment #4) > Comment on attachment 397518 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=397518&action=review > > > Tools/ChangeLog:3 > > + Add some debug output for run-jsc-stress-tests > > Can we embed some of the why in the commit message? Added the radar link as well Created attachment 397736 [details]
Patch
Committed r260786: <https://trac.webkit.org/changeset/260786> All reviewed patches have been landed. Closing bug and clearing flags on attachment 397736 [details]. Comment on attachment 397736 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=397736&action=review This should be behind an option, not the default. This prints each time you run it and it makes it seems like it might have failed. > Tools/Scripts/run-jsc-stress-tests:2165 > + puts "Detacted failures in #{Time.now - start_time}" typo: Detacted => Detected |