HttpCache.RangeGET_ParallelValidationDifferentRanges failing in net_unittests on Fuchsia |
|||
Issue descriptionhttps://build.chromium.org/p/chromium.fyi/builders/Fuchsia/builds/8764 https://build.chromium.org/p/chromium.fyi/builders/Fuchsia/builds/8774 [00146.754] 03070.03119> [ RUN ] HttpCache.RangeGET_ParallelValidationDifferentRanges [00146.754] 03070.03119> ../../net/http/http_cache_unittest.cc:1737: Failure [00146.754] 03070.03119> Expected: 2 [00146.754] 03070.03119> To be equal to: cache.network_layer()->transaction_count() [00146.754] 03070.03119> Which is: 3 [00146.754] 03070.03119> [3:586038165:0815/051311.497046:142176179:ERROR:http_transaction_test_util.cc(534)] Not implemented reached in virtual void net::MockNetworkTransaction::GetConnectionAttempts(net::ConnectionAttempts *) const [00146.754] 03070.03119> ../../net/http/http_cache_unittest.cc:1752: Failure [00146.754] 03070.03119> Expected: 2 [00146.754] 03070.03119> To be equal to: cache.network_layer()->transaction_count() [00146.754] 03070.03119> Which is: 4 [00146.754] 03070.03119> [ FAILED ] HttpCache.RangeGET_ParallelValidationDifferentRanges (202 ms)
,
Aug 16 2017
Apparently I failed at disabling this? https://build.chromium.org/p/chromium.fyi/builders/Fuchsia/builds/8839 flaked again.
,
Aug 16 2017
Oh maybe it doesn't handle end of line comments? https://chromium-review.googlesource.com/c/614867/2/testing/buildbot/filters/fuchsia.net_unittests.filter
,
Aug 16 2017
Yeah, looks like the comment is the problem.
,
Aug 16 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7e4018819b95b94dc1c92820ca14a7dbdb304b24 commit 7e4018819b95b94dc1c92820ca14a7dbdb304b24 Author: Scott Graham <scottmg@chromium.org> Date: Wed Aug 16 20:50:36 2017 fuchsia: move comment in net_unittests filter End of line comments don't seem to work in .filter files. See https://bugs.chromium.org/p/chromium/issues/detail?id=755552#c3. It is documented this way, despite this being a bit weak: https://cs.chromium.org/chromium/src/testing/buildbot/filters/README.md?q=testing/buildbot/filters/README.md&sq=package:chromium&dr Bug: 755552 Change-Id: I337743b63922e347b029bcbdb323da3241585735 Reviewed-on: https://chromium-review.googlesource.com/617644 Commit-Queue: Scott Graham <scottmg@chromium.org> Commit-Queue: Sergey Ulanov <sergeyu@chromium.org> Reviewed-by: Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#494940} [modify] https://crrev.com/7e4018819b95b94dc1c92820ca14a7dbdb304b24/testing/buildbot/filters/fuchsia.net_unittests.filter
,
Aug 18 2017
,
Sep 5 2017
,
Sep 13 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3d720181b21811bcb8e80f49cffc5bca13fd34f0 commit 3d720181b21811bcb8e80f49cffc5bca13fd34f0 Author: Sergey Ulanov <sergeyu@chromium.org> Date: Wed Sep 13 01:21:57 2017 Enable RangeGet_* on Fuchsia to verify if they are still flaky Two RangeGet tests were disabled in net_unittests on Fuchsia because they were flaky. It's likely that these two tests were fixed with http://crrev.com/500757 (which added IO event handling in RunLoop::RunUntilIdle()). Enable them to see if they are still flaky. Bug: 755552 Change-Id: I96146aa7e8247aff764853de656d9997d3cf5028 Reviewed-on: https://chromium-review.googlesource.com/663798 Reviewed-by: Scott Graham <scottmg@chromium.org> Commit-Queue: Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#501500} [modify] https://crrev.com/3d720181b21811bcb8e80f49cffc5bca13fd34f0/testing/buildbot/filters/fuchsia.net_unittests.filter
,
Sep 20 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by bugdroid1@chromium.org
, Aug 15 2017