Bug 165222

Summary: REGRESSION (r209144): imported/w3c/web-platform-tests/IndexedDB/idbindex-* tests are flaky in debug builds
Product: WebKit Reporter: Ryan Haddad <ryanhaddad>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: beidson, jlewis3, sihui_liu
Priority: P2    
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=164974
Bug Depends on:    
Bug Blocks: 154968    

Ryan Haddad
Reported 2016-11-30 14:24:41 PST
The following tests have been flaky since https://trac.webkit.org/changeset/209144 landed. imported/w3c/web-platform-tests/IndexedDB/idbindex-multientry.htm imported/w3c/web-platform-tests/IndexedDB/idbindex-rename-abort.html imported/w3c/web-platform-tests/IndexedDB/idbindex-rename-errors.html https://build.webkit.org/results/Apple%20El%20Capitan%20Debug%20WK2%20(Tests)/r209145%20(9666)/results.html https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=imported%2Fw3c%2Fweb-platform-tests%2FIndexedDB%2Fidbindex-multientry.htm https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=imported%2Fw3c%2Fweb-platform-tests%2FIndexedDB%2Fidbindex-rename-abort.html https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=imported%2Fw3c%2Fweb-platform-tests%2FIndexedDB%2Fidbindex-rename-errors.html --- /Volumes/Data/slave/elcapitan-debug-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/IndexedDB/idbindex-multientry-expected.txt +++ /Volumes/Data/slave/elcapitan-debug-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/IndexedDB/idbindex-multientry-actual.txt @@ -1,3 +1,5 @@ -PASS IDBIndex.multiEntry - adding keys +Harness Error (TIMEOUT), message = null +NOTRUN IDBIndex.multiEntry - adding keys +
Attachments
Ryan Haddad
Comment 1 2016-11-30 14:25:55 PST
This affects macOS Debug WK2
Brady Eidson
Comment 2 2016-11-30 17:03:55 PST
Hammering these in WK2 Debug locally, all sorts of configurations of run-webkit-tests. Cannot reproduce.
Brady Eidson
Comment 3 2016-12-01 13:09:31 PST
Wonder if there's any Database process crashes and they're just not coming up in the results?
Ryan Haddad
Comment 4 2016-12-01 13:10:31 PST
(In reply to comment #3) > Wonder if there's any Database process crashes and they're just not coming > up in the results? I'll see if I can find anything.
Alexey Proskuryakov
Comment 5 2016-12-01 13:18:41 PST
Brady, how long do these tests take when you run them? If they are slow locally, timing out on the bots wouldn't be surprising.
Ryan Haddad
Comment 6 2016-12-01 13:22:14 PST
(In reply to comment #4) > (In reply to comment #3) > > Wonder if there's any Database process crashes and they're just not coming > > up in the results? > > I'll see if I can find anything. I don't see any database process crashlogs on the bot that most recently had these failures.
Brady Eidson
Comment 7 2016-12-01 14:07:46 PST
(In reply to comment #5) > Brady, how long do these tests take when you run them? If they are slow > locally, timing out on the bots wouldn't be surprising. Extremely quickly. With webkit-httpd running, the command: `DYLD_FRAMEWORK_PATH=~/build/Debug/ ~/build/Debug/WebKitTestRunner http://127.0.0.1:8800/IndexedDB/idbindex-multientry.htm http://127.0.0.1:8800/IndexedDB/idbindex-rename-abort.html http://127.0.0.1:8800/IndexedDB/idbindex-rename-errors.html` ...takes well under a second to successfully complete.
Brady Eidson
Comment 8 2016-12-02 14:00:47 PST
According to the flakiness dashboard, these literally never timeout on Release builds. But on Debug builds they take *a long* time. As mentioned in comment 7, I can't reproduce these locally even with a debug build. They are very fast.
Brady Eidson
Comment 9 2016-12-02 14:02:17 PST
I'm also trying to reason through why they could possibly be related to this patch, and I got nothin'
Brady Eidson
Comment 10 2016-12-02 14:03:56 PST
Even completes quickly under gmalloc (locally)
Brady Eidson
Comment 11 2016-12-02 14:06:49 PST
We should mark them flakey for debug only.
Ryan Haddad
Comment 12 2016-12-02 14:43:41 PST
(In reply to comment #11) > We should mark them flakey for debug only. Done in http://trac.webkit.org/projects/webkit/changeset/209272
Sihui Liu
Comment 14 2021-07-30 10:26:40 PDT
Expectations updated at https://bugs.webkit.org/show_bug.cgi?id=228241 as the test is no longer flaky. *** This bug has been marked as a duplicate of bug 228241 ***
Note You need to log in before you can comment on or make changes to this bug.