New issue
Advanced search Search tips

Issue 693810 link

Starred by 4 users

Issue metadata

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



Sign in to add a comment

"ShelfAppBrowserTest.ActivationStateCheck" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Feb 18 2017

Issue description

"ShelfAppBrowserTest.ActivationStateCheck" 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 22 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyMwsSBUZsYWtlIihTaGVsZkFwcEJyb3dzZXJUZXN0LkFjdGl2YXRpb25TdGF0ZUNoZWNrDA.

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, Feb 19 2017

Detected 20 new flakes for test/step "ShelfAppBrowserTest.ActivationStateCheck". To see the actual flakes, please visit https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyMwsSBUZsYWtlIihTaGVsZkFwcEJyb3dzZXJUZXN0LkFjdGl2YXRpb25TdGF0ZUNoZWNrDA. This message was posted automatically by the chromium-try-flakes app.
Project Member

Comment 2 by chromium...@appspot.gserviceaccount.com, Feb 20 2017

Detected 8 new flakes for test/step "ShelfAppBrowserTest.ActivationStateCheck". To see the actual flakes, please visit https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyMwsSBUZsYWtlIihTaGVsZkFwcEJyb3dzZXJUZXN0LkFjdGl2YXRpb25TdGF0ZUNoZWNrDA. This message was posted automatically by the chromium-try-flakes app.
Labels: -Sheriff-Chromium
Owner: steve...@chromium.org
Status: Assigned (was: Untriaged)
->stevenjb since you're an OWNER
Project Member

Comment 4 by bugdroid1@chromium.org, Feb 20 2017

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

commit 84393884d5bd97bc8330fe82f2954ca093ba21d6
Author: maxmorin <maxmorin@chromium.org>
Date: Mon Feb 20 12:47:43 2017

Disable flaky ShelfAppBrowserTest_ActivationStateCheck.

TBR=stevenjb
BUG= 693810 

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

[modify] https://crrev.com/84393884d5bd97bc8330fe82f2954ca093ba21d6/chrome/browser/ui/ash/launcher/chrome_launcher_controller_impl_browsertest.cc

Cc: khmel@chromium.org
Cc: xiy...@chromium.org steve...@chromium.org
Components: UI>Shell>Launcher
Labels: -Pri-1 Pri-2
Owner: khmel@chromium.org
Status: Available (was: Assigned)
I'm extremely unlikely to get to this soon. Lowering the priority because it doesn't seem like a super critical test, but it would be nice to at least investigate the failure...

Project Member

Comment 7 by bugdroid1@chromium.org, Jul 10 2017

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

commit f14e8c5353fef67e28d07666535767c5839c3374
Author: khmel <khmel@google.com>
Date: Mon Jul 10 20:38:35 2017

Re-enable ShelfAppBrowserTest.ActivationStateCheck

This looks like transient error.

Bug:  693810 
Test: Run bot 10 times and checked output log,
      ShelfAppBrowserTest.ActivationStateCheck never failed or retried.

Change-Id: I8bc7edf66d4c07d316bd50b4d8a35730a023a855
Reviewed-on: https://chromium-review.googlesource.com/549615
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Commit-Queue: Yury Khmel <khmel@google.com>
Cr-Commit-Position: refs/heads/master@{#485365}
[modify] https://crrev.com/f14e8c5353fef67e28d07666535767c5839c3374/chrome/browser/ui/ash/launcher/chrome_launcher_controller_browsertest.cc

Comment 8 by khmel@chromium.org, Aug 7 2017

Status: Fixed (was: Available)

Sign in to add a comment