New issue
Advanced search Search tips

Issue 668197 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

"DataReductionProxyInterceptorEndToEndTest.RedirectChainToHttps" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Nov 23 2016

Issue description

"DataReductionProxyInterceptorEndToEndTest.RedirectChainToHttps" 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=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNySQsSBUZsYWtlIj5EYXRhUmVkdWN0aW9uUHJveHlJbnRlcmNlcHRvckVuZFRvRW5kVGVzdC5SZWRpcmVjdENoYWluVG9IdHRwcww.

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
Owner: tbansal@chromium.org
Status: Assigned (was: Untriaged)
tbansal@, could you please take a look?  (you are the last person that touched the file with the unit tests;  I hope you can route the bug to the right person).

Removing this bug from the Sheriff queue - a CL that disables the test is in CQ - see https://codereview.chromium.org/2524083002
Components: Internals>Network>DataProxy
Labels: OS-Android
Owner: ----
Status: Untriaged (was: Assigned)
Changed the component so it gets triaged. @lukasza: thanks for filing the bug.
Project Member

Comment 4 by bugdroid1@chromium.org, Nov 23 2016

Comment 5 by bengr@chromium.org, Dec 22 2016

Labels: -Pri-1 M-57 Pri-2
Owner: tbansal@chromium.org
Status: Assigned (was: Untriaged)

Comment 6 by bengr@chromium.org, May 5 2017

Tarun, any progress? And what's the severity of this lost test coverage?
Labels: -M-57 M-60
No progress yet.
Status: Started (was: Assigned)

Comment 10 Deleted

Re #6: I believe the severity is low since the test is disabled on only Android platform.
Project Member

Comment 12 by bugdroid1@chromium.org, May 12 2017

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

commit 4088d7f84906876ccd2721f313dc21441cbe497f
Author: tbansal <tbansal@chromium.org>
Date: Fri May 12 22:10:32 2017

Fix flaky data reduction proxy test

Attempt to fix the flaky
"DataReductionProxyInterceptorEndToEndTest.RedirectChainToHttps" test.
The CL adds load flags to prevent cache interaction which might
have caused the test to flake in the past.

BUG= 668197 

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

[modify] https://crrev.com/4088d7f84906876ccd2721f313dc21441cbe497f/components/data_reduction_proxy/core/browser/data_reduction_proxy_interceptor_unittest.cc

Status: Fixed (was: Started)

Sign in to add a comment