Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(13)

Issue 2236493003: [catapult android trybot] Make Telemetry tests run on Android (Closed)

Created:
4 years, 4 months ago by charliea (OOO until 10-5)
Modified:
3 years, 11 months ago
CC:
catapult-reviews_chromium.org
Base URL:
git@github.com:catapult-project/catapult.git@master
Target Ref:
refs/heads/master
Project:
catapult
Visibility:
Public.

Description

[catapult android trybot] Make Telemetry tests run on Android BUG=catapult:#2645,catapult:#3099 Review-Url: https://codereview.chromium.org/2236493003 Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/e32c287c50d6315a67079bd40dbf88c85dab3696

Patch Set 1 : Checkpoint. #

Patch Set 2 : Checkpoint. #

Total comments: 2

Patch Set 3 : Merge branch 'master' into fix_android_recipe #

Patch Set 4 : Merge branch 'master' of github.com:catapult-project/catapult into fix_android_recipe #

Patch Set 5 : Sync to head #

Patch Set 6 : Disable failing Android tests #

Patch Set 7 : Disable more tests #

Patch Set 8 : Test to see if isolating all tests fixes problems #

Patch Set 9 : Test to see if --jobs=1 fixes the problem #

Patch Set 10 : Print out debugging information about hostname and port of websocket #

Patch Set 11 : Add stack traces and more logging #

Unified diffs Side-by-side diffs Delta from patch set Stats (+189 lines, -35 lines) Patch
M catapult_build/build_steps.py View 1 2 3 4 2 chunks +12 lines, -1 line 0 comments Download
M telemetry/telemetry/core/memory_cache_http_server_unittest.py View 1 2 3 4 5 2 chunks +3 lines, -0 lines 0 comments Download
M telemetry/telemetry/core/platform_unittest.py View 1 2 3 4 5 6 1 chunk +6 lines, -2 lines 0 comments Download
M telemetry/telemetry/core/tracing_controller_unittest.py View 1 2 3 4 5 5 chunks +10 lines, -1 line 0 comments Download
M telemetry/telemetry/internal/actions/action_runner_unittest.py View 1 2 3 4 5 6 5 chunks +11 lines, -1 line 0 comments Download
M telemetry/telemetry/internal/actions/drag_unittest.py View 1 2 3 4 5 1 chunk +3 lines, -1 line 0 comments Download
M telemetry/telemetry/internal/actions/key_event_unittest.py View 1 2 3 4 5 2 chunks +3 lines, -0 lines 0 comments Download
M telemetry/telemetry/internal/actions/load_media_unittest.py View 1 2 3 4 5 6 2 chunks +6 lines, -2 lines 0 comments Download
M telemetry/telemetry/internal/actions/navigate_unittest.py View 1 2 3 4 5 6 1 chunk +3 lines, -0 lines 0 comments Download
M telemetry/telemetry/internal/actions/page_action_unittest.py View 1 2 3 4 5 1 chunk +3 lines, -0 lines 0 comments Download
M telemetry/telemetry/internal/actions/play_unittest.py View 1 2 3 4 5 1 chunk +3 lines, -2 lines 0 comments Download
M telemetry/telemetry/internal/actions/repeatable_scroll_unittest.py View 1 2 3 4 5 6 2 chunks +4 lines, -0 lines 0 comments Download
M telemetry/telemetry/internal/actions/scroll_unittest.py View 1 2 3 4 5 2 chunks +3 lines, -0 lines 0 comments Download
M telemetry/telemetry/internal/actions/seek_unittest.py View 1 2 3 4 5 6 2 chunks +6 lines, -2 lines 0 comments Download
M telemetry/telemetry/internal/actions/tap_unittest.py View 1 2 3 4 5 1 chunk +3 lines, -1 line 0 comments Download
M telemetry/telemetry/internal/backends/chrome/tab_list_backend_unittest.py View 1 2 3 4 5 1 chunk +2 lines, -0 lines 0 comments Download
M telemetry/telemetry/internal/backends/chrome_inspector/devtools_client_backend_unittest.py View 1 2 3 4 5 6 1 chunk +7 lines, -1 line 0 comments Download
M telemetry/telemetry/internal/backends/chrome_inspector/inspector_memory_unittest.py View 1 2 3 4 5 6 1 chunk +2 lines, -0 lines 0 comments Download
M telemetry/telemetry/internal/backends/chrome_inspector/inspector_page_unittest.py View 1 2 3 4 5 1 chunk +2 lines, -0 lines 0 comments Download
M telemetry/telemetry/internal/backends/chrome_inspector/inspector_runtime_unittest.py View 1 2 3 4 5 2 chunks +3 lines, -0 lines 0 comments Download
M telemetry/telemetry/internal/backends/chrome_inspector/memory_backend_unittest.py View 1 2 3 4 5 2 chunks +3 lines, -1 line 0 comments Download
M telemetry/telemetry/internal/backends/chrome_inspector/tracing_backend_unittest.py View 1 2 3 4 5 6 1 chunk +3 lines, -2 lines 0 comments Download
M telemetry/telemetry/internal/browser/browser_unittest.py View 1 2 3 4 5 6 4 chunks +9 lines, -1 line 0 comments Download
M telemetry/telemetry/internal/browser/tab_unittest.py View 1 2 3 4 5 6 3 chunks +6 lines, -0 lines 0 comments Download
M telemetry/telemetry/internal/browser/user_agent_unittest.py View 1 2 3 4 5 1 chunk +3 lines, -1 line 0 comments Download
M telemetry/telemetry/internal/platform/android_device_unittest.py View 1 2 3 4 5 6 chunks +12 lines, -6 lines 0 comments Download
M telemetry/telemetry/internal/platform/profiler/android_profiling_helper_unittest.py View 1 2 3 4 5 1 chunk +2 lines, -1 line 0 comments Download
M telemetry/telemetry/internal/platform/profiler/android_systrace_profiler_unittest.py View 1 2 3 4 5 1 chunk +2 lines, -1 line 0 comments Download
M telemetry/telemetry/internal/platform/profiler/trace_profiler_unittest.py View 1 2 3 4 5 6 1 chunk +3 lines, -0 lines 0 comments Download
M telemetry/telemetry/page/cache_temperature_unittest.py View 1 2 3 4 5 1 chunk +2 lines, -0 lines 0 comments Download
M telemetry/telemetry/page/page_run_end_to_end_unittest.py View 1 2 3 4 5 6 8 chunks +16 lines, -0 lines 0 comments Download
M telemetry/telemetry/page/page_test_unittest.py View 1 2 3 4 5 1 chunk +2 lines, -0 lines 0 comments Download
M telemetry/telemetry/record_wpr_unittest.py View 1 2 3 4 5 6 5 chunks +11 lines, -1 line 0 comments Download
M telemetry/telemetry/testing/run_tests.py View 1 2 3 4 5 6 7 8 1 chunk +1 line, -2 lines 0 comments Download
M telemetry/telemetry/web_perf/smooth_gesture_util_unittest.py View 1 2 3 4 5 1 chunk +3 lines, -1 line 0 comments Download
M telemetry/telemetry/web_perf/timeline_based_page_test_unittest.py View 1 2 3 4 5 4 chunks +8 lines, -4 lines 0 comments Download
M telemetry/third_party/websocket-client/websocket.py View 1 2 3 4 5 6 7 8 9 10 3 chunks +8 lines, -0 lines 0 comments Download

Messages

Total messages: 130 (97 generated)
nednguyen
lgtm
4 years, 4 months ago (2016-08-10 14:01:23 UTC) #5
aiolos (Not reviewing)
https://codereview.chromium.org/2236493003/diff/40001/catapult_build/build_steps.py File catapult_build/build_steps.py (right): https://codereview.chromium.org/2236493003/diff/40001/catapult_build/build_steps.py#newcode137 catapult_build/build_steps.py:137: '--start-xvfb' Do we not need to start the xvfb ...
4 years, 4 months ago (2016-08-11 16:36:11 UTC) #19
nednguyen
https://codereview.chromium.org/2236493003/diff/40001/catapult_build/build_steps.py File catapult_build/build_steps.py (right): https://codereview.chromium.org/2236493003/diff/40001/catapult_build/build_steps.py#newcode137 catapult_build/build_steps.py:137: '--start-xvfb' On 2016/08/11 16:36:10, aiolos wrote: > Do we ...
4 years, 4 months ago (2016-08-11 16:37:17 UTC) #20
aiolos (Not reviewing)
On 2016/08/11 16:37:17, nednguyen wrote: > https://codereview.chromium.org/2236493003/diff/40001/catapult_build/build_steps.py > File catapult_build/build_steps.py (right): > > https://codereview.chromium.org/2236493003/diff/40001/catapult_build/build_steps.py#newcode137 > ...
4 years, 4 months ago (2016-08-11 17:11:52 UTC) #21
nednguyen
On 2016/08/11 17:11:52, aiolos wrote: > On 2016/08/11 16:37:17, nednguyen wrote: > > > https://codereview.chromium.org/2236493003/diff/40001/catapult_build/build_steps.py ...
4 years, 4 months ago (2016-08-11 17:13:51 UTC) #23
aiolos (Not reviewing)
> > Oh, it's possible that our list of dependencies to pre-fetch for android is ...
4 years, 4 months ago (2016-08-11 18:02:05 UTC) #25
aiolos (Not reviewing)
lgtm. I'm landing.
4 years, 4 months ago (2016-08-15 21:00:10 UTC) #26
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2236493003/40001
4 years, 4 months ago (2016-08-15 21:00:27 UTC) #28
commit-bot: I haz the power
Try jobs failed on following builders: Catapult Android Tryserver on master.tryserver.client.catapult (JOB_FAILED, https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Android%20Tryserver/builds/831)
4 years, 4 months ago (2016-08-15 21:41:28 UTC) #30
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2236493003/40001
4 years, 4 months ago (2016-08-17 14:15:18 UTC) #32
commit-bot: I haz the power
Try jobs failed on following builders: Catapult Android Tryserver on master.tryserver.client.catapult (JOB_FAILED, https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Android%20Tryserver/builds/863)
4 years, 4 months ago (2016-08-17 15:00:36 UTC) #34
jbudorick
Rolled devil's dependencies. The device_forwarder should no longer complain about ASAN, so I'm trying this ...
4 years, 4 months ago (2016-08-17 20:39:32 UTC) #36
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2236493003/40001
4 years, 4 months ago (2016-08-17 20:39:41 UTC) #38
commit-bot: I haz the power
Try jobs failed on following builders: Catapult Android Tryserver on master.tryserver.client.catapult (JOB_FAILED, https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Android%20Tryserver/builds/874)
4 years, 4 months ago (2016-08-17 21:08:25 UTC) #40
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2236493003/40001
4 years, 4 months ago (2016-08-19 14:41:33 UTC) #42
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2236493003/60001
4 years, 4 months ago (2016-08-19 14:43:23 UTC) #46
commit-bot: I haz the power
Try jobs failed on following builders: Catapult Android Tryserver on master.tryserver.client.catapult (JOB_FAILED, https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Android%20Tryserver/builds/923)
4 years, 4 months ago (2016-08-19 15:09:08 UTC) #48
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2236493003/60001
4 years, 3 months ago (2016-08-29 14:32:43 UTC) #58
commit-bot: I haz the power
Try jobs failed on following builders: Catapult Android Tryserver on master.tryserver.client.catapult (JOB_FAILED, https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Android%20Tryserver/builds/1098) Catapult Presubmit ...
4 years, 3 months ago (2016-08-29 14:33:53 UTC) #60
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2236493003/80001
4 years, 3 months ago (2016-08-29 18:11:18 UTC) #63
commit-bot: I haz the power
Try jobs failed on following builders: Catapult Linux Tryserver on master.tryserver.client.catapult (JOB_FAILED, https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Linux%20Tryserver/builds/4642)
4 years, 3 months ago (2016-08-29 18:26:36 UTC) #65
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2236493003/80001
4 years, 3 months ago (2016-08-31 15:34:19 UTC) #67
commit-bot: I haz the power
Try jobs failed on following builders: Catapult Android Tryserver on master.tryserver.client.catapult (JOB_FAILED, https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Android%20Tryserver/builds/1155)
4 years, 3 months ago (2016-08-31 16:11:53 UTC) #69
nednguyen
On 2016/08/31 16:11:53, commit-bot: I haz the power wrote: > Try jobs failed on following ...
4 years, 3 months ago (2016-09-01 23:00:17 UTC) #70
aiolos (Not reviewing)
On 2016/09/01 23:00:17, nednguyen wrote: > On 2016/08/31 16:11:53, commit-bot: I haz the power wrote: ...
4 years, 3 months ago (2016-09-02 18:16:02 UTC) #71
charliea (OOO until 10-5)
After the first round of disables, it looks like there's some more flaky tests to ...
4 years ago (2016-12-19 16:24:34 UTC) #84
nednguyen
lgtm
4 years ago (2016-12-19 16:28:12 UTC) #85
charliea (OOO until 10-5)
Trying another dry run after having pschmidt@ restart the host (https://bugs.chromium.org/p/chromium/issues/detail?id=675743).
4 years ago (2016-12-20 16:56:03 UTC) #91
nednguyen
On 2016/12/20 16:56:03, charliea wrote: > Trying another dry run after having pschmidt@ restart the ...
4 years ago (2016-12-20 18:21:33 UTC) #96
charliea (OOO until 10-5)
> Can you add "--jobs=1" to the test command line see if it fixes all ...
4 years ago (2016-12-20 18:45:05 UTC) #103
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2236493003/250001
3 years, 11 months ago (2017-01-04 22:38:59 UTC) #126
commit-bot: I haz the power
Committed patchset #11 (id:250001) as https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/e32c287c50d6315a67079bd40dbf88c85dab3696
3 years, 11 months ago (2017-01-04 22:41:17 UTC) #129
nednguyen
3 years, 11 months ago (2017-01-05 03:00:25 UTC) #130
Message was sent while issue was closed.
On 2017/01/04 22:41:17, commit-bot: I haz the power wrote:
> Committed patchset #11 (id:250001) as
>
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

Hmhh, shouldn't you undo the change to disabled tests as the root cause is
figured out?

Powered by Google App Engine
This is Rietveld 408576698