New issue
Advanced search Search tips

Issue 878650 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 29
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

"inspector-protocol/heap-profiler/heap-snapshot-merged-nodes.js" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Aug 29

Issue description

"inspector-protocol/heap-profiler/heap-snapshot-merged-nodes.js" 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=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNySQsSBUZsYWtlIj5pbnNwZWN0b3ItcHJvdG9jb2wvaGVhcC1wcm9maWxlci9oZWFwLXNuYXBzaG90LW1lcmdlZC1ub2Rlcy5qcww.

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
 
This test is timing out.

We don't see similar failures on the main waterfall:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_layout_tests&tests=inspector-protocol%2Fheap-profiler%2Fheap-snapshot-merged-nodes.js

The test seems consistently slow on any platforms, so we probably should
make this test Slow on release builds, too.
I'm making this test Slow.
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 29

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

commit 389f1969a660996219e02dc133acd5198e0ab846
Author: Yuta Kitamura <yutak@chromium.org>
Date: Wed Aug 29 09:08:30 2018

inspector-protocol/heap-profiler/heap-snapshot-merged-nodes.js is slow.

This test seems slow even on release bots.

TBR=nednguyen@chromium.org

Bug:  878650 
Change-Id: I56377cff5409adfa0dc1b580650ca2f76939fd0c
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/1195125
Reviewed-by: Yuta Kitamura <yutak@chromium.org>
Commit-Queue: Yuta Kitamura <yutak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#587044}
[modify] https://crrev.com/389f1969a660996219e02dc133acd5198e0ab846/third_party/WebKit/LayoutTests/SlowTests

Labels: -Pri-1 Pri-2
Now this test has a longer timeout. Let's see if this works.
Owner: yutak@chromium.org
Status: Fixed (was: Untriaged)
Seems like there is no more failure
Project Member

Comment 6 by bugdroid1@chromium.org, Aug 31

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

commit 79c4f8ad40308f2274c9c9996a0acf3d712fe719
Author: Xianzhu Wang <wangxianzhu@chromium.org>
Date: Fri Aug 31 23:18:28 2018

inspector-protocol/heap-profiler/heap-snapshot-with-no-detached-iframe.js is slow

especially on CQ bots (which has DCHECK enabled):
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_layout_tests%20(with%20patch)&tests=inspector-protocol%2Fheap-profiler%2Fheap-snapshot-with-no-detached-iframe.js

TBR=wangxianzhu@chromium.org

Bug:  878650 
Change-Id: Ibdcaa931c8820e9dfcf2f6c3e88922aa7345dc5b
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/1199720
Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#588206}
[modify] https://crrev.com/79c4f8ad40308f2274c9c9996a0acf3d712fe719/third_party/WebKit/LayoutTests/SlowTests

Sign in to add a comment