CronetUrlRequestContextTest#testRealTimeNetworkQualityObservations failing on all Cronet bots |
||||
Issue descriptionExample failure: https://uberchromegw.corp.google.com/i/chromium.android/builders/Android%20Cronet%20Builder%20(dbg)/builds/2059 . I can also repro locally, seems like a recent regression.
,
May 24 2016
Tarun, would you mind taking a look at this? Do you know if there's any recent change in NQE that might have caused this test failure?
,
May 24 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/aeee178ce621bfa3a9250f749d8e54a5b42bc54d commit aeee178ce621bfa3a9250f749d8e54a5b42bc54d Author: aelias <aelias@chromium.org> Date: Tue May 24 01:37:49 2016 Mark CronetUrlRequestContextTest#testRealTimeNetworkQualityObservations flaky. This test failed once on 3 different Cronet bots, and I can also reproduce flakiness locally. TBR=xunjieli NOTRY=true BUG= 614227 Review-Url: https://codereview.chromium.org/2005153002 Cr-Commit-Position: refs/heads/master@{#395508} [modify] https://crrev.com/aeee178ce621bfa3a9250f749d8e54a5b42bc54d/components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlRequestContextTest.java
,
May 24 2016
I noticed this before, and I have verified that https://codereview.chromium.org/1999303002/ fixes it. I manually added android_cronet_tester on that CQ, and it looks happy. Can you look at the CL, ad if it looks good, we can push it to CQ?
,
May 27 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/86d66363a9aad84556298c81c4a5af935d67d03b commit 86d66363a9aad84556298c81c4a5af935d67d03b Author: tbansal <tbansal@chromium.org> Date: Fri May 27 16:42:17 2016 Fix flaky NQE cronet test Example of a flaky run from testbot: https://build.chromium.org/p/tryserver.chromium.android/builders/android_cronet_tester/builds/168 and https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20Builder/builds/2184 A recent NQE CL changed the logic so that throughput observations are now posted to NQE (before that CL, the observations were available synchronously). BUG= 614227 Review-Url: https://codereview.chromium.org/1999303002 Cr-Commit-Position: refs/heads/master@{#396488} [modify] https://crrev.com/86d66363a9aad84556298c81c4a5af935d67d03b/components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlRequestContextTest.java
,
May 27 2016
,
Feb 14 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by aelias@chromium.org
, May 24 2016Owner: xunji...@chromium.org