Bug 262898

Summary: run-webkit-tests: check for crash log without leading period
Product: WebKit Reporter: David Kilzer (:ddkilzer) <ddkilzer>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: ap, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Other   
Hardware: Unspecified   
OS: Unspecified   

Description David Kilzer (:ddkilzer) 2023-10-09 10:23:37 PDT
run-webkit-tests: check for crash log without leading period.

When ReportCrash writes out a crash log, the initial file name contains a leading period ("." or dot) character, then the file is renamed to the same file name without the leading period when it's done being written.

The run-webkit-tests tool sometimes finds the crash log with the leading period in the name, but then can't make a copy of the crash log because the file was renamed before it copies it.

Example:

PR https://github.com/WebKit/WebKit/pull/18195
EWS https://ews-build.webkit.org/#/builders/10/builds/19515
Layout test result https://ews-build.s3-us-west-2.amazonaws.com/macOS-Monterey-Release-WK1-Tests-EWS/71cbcf72-19515/results.html
Comment 1 David Kilzer (:ddkilzer) 2023-10-09 10:24:46 PDT
One way to avoid missing crash logs is to check (once) for the same crash log without the leading period character when copying the file if it can't find it the first time.
Comment 2 Radar WebKit Bug Importer 2023-10-09 10:25:37 PDT
<rdar://problem/116687173>
Comment 3 Alexey Proskuryakov 2023-10-09 10:28:23 PDT
I don't think that this is the right thing to do long term. We will still have a race with CrashTracer, as it generates the crash report.

What we need is to monitor the state of CrashTracer to get all the necessary reports.
Comment 4 David Kilzer (:ddkilzer) 2023-10-09 10:36:07 PDT
Yep, it's a workaround.  But it would help with a certain subset of missing crash logs in the meantime.