New issue
Advanced search Search tips

Issue 708175 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

"external/wpt/IndexedDB/interleaved-cursors.html" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Apr 4 2017

Issue description

"external/wpt/IndexedDB/interleaved-cursors.html" is flaky.

This issue was created automatically by the chromium-try-flakes app. Please find the right owner to fix the respective test/step and assign this issue to them. If the step/test is infrastructure-related, please add Infra-Troopers label and change issue status to Untriaged. When done, please remove the issue from Sheriff Bug Queue by removing the Sheriff-Chromium label.

We have detected 3 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyOgsSBUZsYWtlIi9leHRlcm5hbC93cHQvSW5kZXhlZERCL2ludGVybGVhdmVkLWN1cnNvcnMuaHRtbAw.

Flaky tests should be disabled within 30 minutes unless culprit CL is found and reverted. Please see more details here: https://sites.google.com/a/chromium.org/dev/developers/tree-sheriffs/sheriffing-bug-queues#triaging-auto-filed-flakiness-bugs
 
Labels: -Sheriff-Chromium
https://codereview.chromium.org/2803563002/ should've had this in the BUG field.
Status: Verified (was: Untriaged)

Comment 3 by creis@chromium.org, Apr 7 2017

Cc: dmu...@chromium.org
Owner: pwnall@chromium.org
Status: Assigned (was: Verified)
maxmorin's CL from comment 1 only reverted the test to an earlier version, after a similar interleaved-overlapping-cursors.html test in r461932 started failing.

However, the current version of the interleaved-cursors.html test is still flaky.  It just failed on WebKit Linux Trusty MSAN here:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty%20MSAN/builds/962

07:53:09.638 7076 worker/3 external/wpt/IndexedDB/interleaved-cursors.html output stderr lines:
07:53:09.638 7076   Xlib:  extension "RANDR" missing on display ":9".
07:53:09.672 6960 [16628/48930] external/wpt/IndexedDB/interleaved-cursors.html failed unexpectedly (test timed out)
07:53:09.639 7076 worker/3 killing primary driver
07:53:09.639 7076 worker/3 killing secondary driver
07:53:09.639 7076 worker/3 external/wpt/IndexedDB/interleaved-cursors.html failed:
07:53:09.639 7076 worker/3  test timed out

It passed again in build 963, but this shows the test is still having problems.  pwnall@, it looks like you added it recently in r459613.  Can you take a look and fix or disable?  Thanks!

Comment 4 by tkent@chromium.org, May 17 2017

Components: Blink>Storage>IndexedDB
Project Member

Comment 6 by bugdroid1@chromium.org, May 17 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5e6a740eee14d5d62c00723774051409fc9931df

commit 5e6a740eee14d5d62c00723774051409fc9931df
Author: tkent <tkent@chromium.org>
Date: Wed May 17 01:07:53 2017

Mark external/wpt/IndexedDB/interleaved-cursors.html flaky.

BUG= 708175 
TBR=pwnall@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2893513002
Cr-Commit-Position: refs/heads/master@{#472278}

[modify] https://crrev.com/5e6a740eee14d5d62c00723774051409fc9931df/third_party/WebKit/LayoutTests/TestExpectations

Comment 7 by jsb...@chromium.org, May 22 2017

Labels: Pri-2
Project Member

Comment 9 by bugdroid1@chromium.org, Jun 23 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0fccf73b911480b60b55889bb479efdd047e0161

commit 0fccf73b911480b60b55889bb479efdd047e0161
Author: Xida Chen <xidachen@chromium.org>
Date: Fri Jun 23 20:40:49 2017

Mark external/wpt/IndexedDB/interleaved-cursors.html on Linux MSAN

TBR=imcheng@chromium.org
NOTRY=true

Bug:  708175 
Change-Id: I30a103b5458d43b26f48273ab255b1f0615568a8
Reviewed-on: https://chromium-review.googlesource.com/546136
Reviewed-by: Xida Chen <xidachen@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#482016}
[modify] https://crrev.com/0fccf73b911480b60b55889bb479efdd047e0161/third_party/WebKit/LayoutTests/MSANExpectations

Project Member

Comment 10 by bugdroid1@chromium.org, Nov 9 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7c78ce6703f35662023778f9ee555ca14aa745e4

commit 7c78ce6703f35662023778f9ee555ca14aa745e4
Author: Tien-Ren Chen <trchen@chromium.org>
Date: Thu Nov 09 23:21:46 2017

[SPv2] Update flag-specific test expectation

BUG= 708175 , 779366 , 714962 

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I0d60874bb3df80d7d4c288c1c1dfae544e125080
Reviewed-on: https://chromium-review.googlesource.com/758975
Commit-Queue: Philip Rogers <pdr@chromium.org>
Reviewed-by: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515341}
[modify] https://crrev.com/7c78ce6703f35662023778f9ee555ca14aa745e4/third_party/WebKit/LayoutTests/FlagExpectations/enable-slimming-paint-v2
[modify] https://crrev.com/7c78ce6703f35662023778f9ee555ca14aa745e4/third_party/WebKit/LayoutTests/SlowTests
[modify] https://crrev.com/7c78ce6703f35662023778f9ee555ca14aa745e4/third_party/WebKit/LayoutTests/TestExpectations

pwnall@ - let's disable this test until we can spend the time to rework it.
Status: Started (was: Assigned)
CL under review: https://crrev.com/c/908248
Project Member

Comment 15 by bugdroid1@chromium.org, Feb 8 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f41591af81be1182393a2000bacb16ad6d646762

commit f41591af81be1182393a2000bacb16ad6d646762
Author: Victor Costan <pwnall@chromium.org>
Date: Thu Feb 08 17:03:56 2018

IndexedDB WPT: Break down interleaved-cursors test.

The test is currently flaky on the CQ (see associated bug) due to timing
out. This CL breaks down the test in a -small and a -large file. The
size of the largest test is also reduced from 500 cursors to 250
cursors. The two changes combined should greatly reduce the chance of
timeouts.

Bug:  708175 
Change-Id: Ic58c59740a0176e6bc5e6e87b66c5d772a67526a
Reviewed-on: https://chromium-review.googlesource.com/908248
Commit-Queue: Joshua Bell <jsbell@chromium.org>
Reviewed-by: Joshua Bell <jsbell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#535411}
[rename] https://crrev.com/f41591af81be1182393a2000bacb16ad6d646762/third_party/WebKit/LayoutTests/external/wpt/IndexedDB/interleaved-cursors-common.js
[add] https://crrev.com/f41591af81be1182393a2000bacb16ad6d646762/third_party/WebKit/LayoutTests/external/wpt/IndexedDB/interleaved-cursors-large.html
[add] https://crrev.com/f41591af81be1182393a2000bacb16ad6d646762/third_party/WebKit/LayoutTests/external/wpt/IndexedDB/interleaved-cursors-small.html

Project Member

Comment 16 by bugdroid1@chromium.org, Feb 9 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7efcecffec688f0e5f38f83713ad7d3707685b41

commit 7efcecffec688f0e5f38f83713ad7d3707685b41
Author: Victor Costan <pwnall@chromium.org>
Date: Fri Feb 09 04:18:10 2018

IndexedDB WPT: Update failing expectations for interleaved-cursor tests.

The test has been broken down into 2 smaller tests that should pass
everywhere. The tests are still too slow for MSAN.

Bug:  708175 
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_layout_ng;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Ia65e7802c8da93cc12ea175caf223e12f09374d7
Reviewed-on: https://chromium-review.googlesource.com/910389
Reviewed-by: Megan Jablonski <megjablon@chromium.org>
Commit-Queue: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#535643}
[modify] https://crrev.com/7efcecffec688f0e5f38f83713ad7d3707685b41/third_party/WebKit/LayoutTests/FlagExpectations/disable-blink-features=RootLayerScrolling
[modify] https://crrev.com/7efcecffec688f0e5f38f83713ad7d3707685b41/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[modify] https://crrev.com/7efcecffec688f0e5f38f83713ad7d3707685b41/third_party/WebKit/LayoutTests/FlagExpectations/enable-slimming-paint-v2
[modify] https://crrev.com/7efcecffec688f0e5f38f83713ad7d3707685b41/third_party/WebKit/LayoutTests/MSANExpectations
[modify] https://crrev.com/7efcecffec688f0e5f38f83713ad7d3707685b41/third_party/WebKit/LayoutTests/TestExpectations

Fixed?
Status: Fixed (was: Started)

Sign in to add a comment