New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 832798 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 829310



Sign in to add a comment

"DirectoryTreeContextMenu/FileManagerBrowserTest.Test/10" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Apr 13 2018

Issue description

"DirectoryTreeContextMenu/FileManagerBrowserTest.Test/10" 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=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyQgsSBUZsYWtlIjdEaXJlY3RvcnlUcmVlQ29udGV4dE1lbnUvRmlsZU1hbmFnZXJCcm93c2VyVGVzdC5UZXN0LzEwDA.

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
 
Project Member

Comment 1 by chromium...@appspot.gserviceaccount.com, Apr 16 2018

Detected 3 new flakes for test/step "DirectoryTreeContextMenu/FileManagerBrowserTest.Test/10". To see the actual flakes, please visit https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyQgsSBUZsYWtlIjdEaXJlY3RvcnlUcmVlQ29udGV4dE1lbnUvRmlsZU1hbmFnZXJCcm93c2VyVGVzdC5UZXN0LzEwDA. This message was posted automatically by the chromium-try-flakes app.
Labels: -Sheriff-Chromium
Owner: fukino@chromium.org
Status: Assigned (was: Untriaged)
Disabling: https://chromium-review.googlesource.com/c/chromium/src/+/1015003
Project Member

Comment 3 by bugdroid1@chromium.org, Apr 17 2018

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

commit 02dcc3909d88d832e67d10b588867124610115a3
Author: Rune Lillesveen <futhark@chromium.org>
Date: Tue Apr 17 14:48:15 2018

Disable flaky tests in file_manager_browsertest.cc

TBR=fukino@chromium.org

Bug:  833049 ,  832192 ,  833752 ,  832799 ,  832798 
Change-Id: I7d3316f2deae4f0e415e41a95211ee1f105cdf32
Reviewed-on: https://chromium-review.googlesource.com/1015003
Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Commit-Queue: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551331}
[modify] https://crrev.com/02dcc3909d88d832e67d10b588867124610115a3/chrome/browser/chromeos/file_manager/file_manager_browsertest.cc

Comment 4 by noel@chromium.org, Apr 17 2018

Cc: yamaguchi@chromium.org mtomasz@chromium.org noel@chromium.org
None of DirectoryTreeContextMenu/FileManagerBrowserTest.Test* are flaking after the fix landed on http://crrev.com/551279
 
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=browser_tests%20(with%20patch)&tests=DirectoryTreeContextMenu%2FFileManagerBrowserTest.Test

Think the disable in #3 was done based on the data in the OP?  Seems that data is out-of-date.

Comment 5 by noel@chromium.org, Apr 17 2018

Cc: futhark@chromium.org
+futhark@chromium.org, see #4 I guess.
Owner: noel@chromium.org

Comment 7 by noel@chromium.org, Apr 18 2018

Cc: dpranke@chromium.org kbr@chromium.org
Components: Platform>Apps>FileManager
Per #4, this crashing test fixed by http://crrev.com/551279.

Reading patch #3 no reference to in that test, this is a stroke of luck.
  DirectoryTreeContextMenu/FileManagerBrowserTest.Test/10

Thus, DirectoryTreeContextMenu/FileManagerBrowserTest.Test/10 is active and just looking at the flakiness dash board, there have been no CHECK external_backend->CanHandleType(foo bar) failures in the last 24 hours.

Other bots: mus_browser_tests (with patch) also looking good, no CanHandleType crashes evident anymore on the flakiness dashboard in recent runs.

What is evident: right now

https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyQgsSBUZsYWtlIjdEaXJlY3RvcnlUcmVlQ29udGV4dE1lbnUvRmlsZU1hbmFnZXJCcm93c2VyVGVzdC5UZXN0LzEwDA.

is reporting 3 recent CanHandleType crashes for this test.  Looking at the logs, they do not have the fix from http://crrev.com/551279 yet.

Summary: we can chill here as the fix finds its way into our bots/systems. No more action needed on this bug for now.

Comment 8 by noel@chromium.org, Apr 18 2018

Labels: -Pri-1 Pri-3

Comment 9 by noel@chromium.org, Apr 18 2018

9 hours later, still no evidence of CanHandleType crashes, looks to be fixed.

Comment 10 by noel@chromium.org, Apr 18 2018

Status: Fixed (was: Assigned)
Re-enabled DirectoryTreeContextMenu/FileManagerBrowserTest.Test/10 in https://crrev.com/551659

Comment 11 by noel@chromium.org, Apr 18 2018

Blockedon: 829310

Sign in to add a comment