testRepeatableScrollActionNoRepeats is flaky |
|||||
Issue descriptionThe test testRepeatableScrollActionNoRepeats flaked several times recently on the linux-chromeos-rel [1] builder Most recent failure: https://ci.chromium.org/buildbot/chromium.chromiumos/linux-chromeos-rel/6340 0/295] telemetry.internal.actions.repeatable_scroll_unittest.RepeatableScrollActionTest.testRepeatableScrollActionNoRepeats failed unexpectedly 4.6090s: Chrome build location for linux_x86_64 not found. Browser will be run without Flash. Traceback (most recent call last): File "/b/s/w/ir/third_party/catapult/telemetry/telemetry/testing/browser_test_case.py", line 39, in WrappedMethod method(self) File "/b/s/w/ir/third_party/catapult/telemetry/telemetry/internal/actions/repeatable_scroll_unittest.py", line 52, in testRepeatableScrollActionNoRepeats expected_scroll)) AssertionError: scroll_position=102;expected 302 Please fix or disable. [1]: https://ci.chromium.org/buildbot/chromium.chromiumos/linux-chromeos-rel/
,
Mar 29 2018
Test was disabled here: https://chromium.googlesource.com/catapult/+/b1c9ad5aa1d025c93dd71880a84c2381909584d5
,
Apr 23 2018
It looks like the test is still running and flaking out on linux-chromeos-rel: https://ci.chromium.org/buildbot/chromium.chromiumos/linux-chromeos-rel/7179 Should be the test be disabled for 'linux' rather than 'chromeos' (given that these tests run linux build with target_os = 'chromeos' rather than on a Chrome OS device/VM)?
,
Apr 24 2018
Let's do that with a comment saying that this is for linux-chromeos-rel
,
Apr 24 2018
The initial patch was correct: https://chromium-review.googlesource.com/c/catapult/+/984339/1..4
,
Apr 24 2018
https://chromium-review.googlesource.com/c/catapult/+/1024834 Ned - who is the owner of repeatable_scroll? It shouldn't be hard to fix this for linux-chromeos. You have to build chrome with gn arg chromeos=1
,
Apr 24 2018
#6: alexclarke created repeatable_scroll in the first place according to git log
,
Apr 24 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6bafd3b9e0685d0890b8c3f23be67916526daafe commit 6bafd3b9e0685d0890b8c3f23be67916526daafe Author: Achuith Bhandarkar <achuith@chromium.org> Date: Tue Apr 24 17:26:48 2018 Disable testRepeatableScrollActionNoRepeats on linux-chromeos-rel. BUG=chromium:826527 Change-Id: I710b1712e016cd482c233fb930264128016c88bd Reviewed-on: https://chromium-review.googlesource.com/1025102 Reviewed-by: Ned Nguyen <nednguyen@google.com> Reviewed-by: John Budorick <jbudorick@chromium.org> Commit-Queue: Achuith Bhandarkar <achuith@chromium.org> Cr-Commit-Position: refs/heads/master@{#553189} [modify] https://crrev.com/6bafd3b9e0685d0890b8c3f23be67916526daafe/testing/buildbot/chromium.chromiumos.json [modify] https://crrev.com/6bafd3b9e0685d0890b8c3f23be67916526daafe/testing/buildbot/chromium.fyi.json [modify] https://crrev.com/6bafd3b9e0685d0890b8c3f23be67916526daafe/testing/buildbot/test_suites.pyl
,
Apr 25 2018
Also created a YAQS entry for future gardeners: https://yaqs.googleplex.com/eng/q/4644305569579008
,
Apr 25 2018
Reassigning to Alex who can hopefully look at the underlying issue at some point. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by afakhry@chromium.org
, Mar 27 2018