New issue
Advanced search Search tips

Issue 875908 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

"AppViewTest.TestAppViewRefusedDataShouldFail" is flaky

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

Issue description

"AppViewTest.TestAppViewRefusedDataShouldFail" 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=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyNwsSBUZsYWtlIixBcHBWaWV3VGVzdC5UZXN0QXBwVmlld1JlZnVzZWREYXRhU2hvdWxkRmFpbAw.

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 bugdroid1@chromium.org, Aug 20

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

commit 40c7d60f0a63fb2a119cc43f90f0caaf8a944480
Author: Jennifer Apacible <apacible@chromium.org>
Date: Mon Aug 20 18:40:01 2018

Disable SequenceManagerFuzzerProcessorTest test on Linux.

NOTRY=true (disabling test)

TBR: lfg@chromium.org
Bug: 875908
Change-Id: I39e833a40ecf50884088cb5de84a9921bec0c06a
Reviewed-on: https://chromium-review.googlesource.com/1181775
Reviewed-by: apacible <apacible@chromium.org>
Commit-Queue: apacible <apacible@chromium.org>
Cr-Commit-Position: refs/heads/master@{#584511}
[modify] https://crrev.com/40c7d60f0a63fb2a119cc43f90f0caaf8a944480/extensions/browser/guest_view/app_view/app_view_apitest.cc

Labels: -Sheriff-Chromium
Owner: farahcharab@chromium.org
Status: Assigned (was: Untriaged)
Labels: Sheriff-Chromium
Owner: apaci...@chromium.org
The patch or the failure have nothing to do with SequenceManagerFuzzerProcessorTest. Could you please make sure that your test descriptions are correct and the right owners are assigned?
Components: Platform>Apps>BrowserTag
Labels: -Pri-1 -Sheriff-Chromium Pri-2
Owner: ----
Status: Untriaged (was: Assigned)
Owner: wjmaclean@chromium.org
Status: Assigned (was: Untriaged)
Disabling test and removing sheriff label.

wjmaclean@chromium.org: Please take a look or help find a better owner.
Project Member

Comment 6 by bugdroid1@chromium.org, Aug 21

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

commit 3fc6763d4526cd70d04c6618d2863bbc7187eaf3
Author: Guido Urdaneta <guidou@chromium.org>
Date: Tue Aug 21 14:14:26 2018

Disable flaky AppViewTest.TestAppViewRefusedDataShouldFail

The test has flaked on Windows, Linux and Mac.

Bug: 875908
TBR: lazyboy@chromium.org
Change-Id: I214ecaea3d96bd5d2e892556b0f92d7bb4f652d6
Reviewed-on: https://chromium-review.googlesource.com/1183433
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Commit-Queue: Guido Urdaneta <guidou@chromium.org>
Cr-Commit-Position: refs/heads/master@{#584731}
[modify] https://crrev.com/3fc6763d4526cd70d04c6618d2863bbc7187eaf3/chrome/browser/apps/guest_view/app_view_browsertest.cc

Labels: Sheriff-Chromium
Detected 5 new flakes for test/step "AppViewTest.TestAppViewRefusedDataShouldFail". To see the actual flakes, please visit https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyNwsSBUZsYWtlIixBcHBWaWV3VGVzdC5UZXN0QXBwVmlld1JlZnVzZWREYXRhU2hvdWxkRmFpbAw. This message was posted automatically by the chromium-try-flakes app. Since flakiness is ongoing, the issue was moved back into Sheriff Bug Queue (unless already there).
Labels: -Sheriff-Chromium

Sign in to add a comment