local_device_environment doesn't handle exceptions appropriately in SetUp |
|||
Issue descriptionsee attached. We fail to start the logcat monitor on a single device & the entire run crashes. We should instead simply blacklist the device.
,
Jul 13 2016
,
Jul 13 2016
(sorry, already started working on it & neglected to update the status)
,
Jul 13 2016
to clarify: I'm working on fixing local_device_environment, not on how DeviceUtils.HealthyDevices handles checking devices when it has a device_arg.
,
Jul 14 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/18fac5c4bf292cc711aee969b9a88f8d8630e861 commit 18fac5c4bf292cc711aee969b9a88f8d8630e861 Author: catapult-deps-roller <catapult-deps-roller@chromium.org> Date: Thu Jul 14 01:24:43 2016 Roll src/third_party/catapult/ 7c71d1cba..70cc6874b (4 commits). https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/7c71d1cba536..70cc6874be5a $ git log 7c71d1cba..70cc6874b --date=short --no-merges --format='%ad %ae %s' BUG= 627939 TBR=catapult-sheriff@chromium.org Review-Url: https://codereview.chromium.org/2150773002 Cr-Commit-Position: refs/heads/master@{#405387} [modify] https://crrev.com/18fac5c4bf292cc711aee969b9a88f8d8630e861/DEPS
,
Jul 21 2016
Still happening: https://chromium-swarm.appspot.com/user/task/30270b4b864bab11
,
Jul 21 2016
Yeah, I was waiting on the catapult roll last week & forgot about the CL this week. Landing now.
,
Jul 25 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/84c46fae5808a0a2ac1a851e51208fa9343eaafb commit 84c46fae5808a0a2ac1a851e51208fa9343eaafb Author: jbudorick <jbudorick@chromium.org> Date: Mon Jul 25 21:43:49 2016 [Android] Blacklist devices on failures during environment set up + tear down. Requires https://codereview.chromium.org/2144103002/ BUG= 627939 Review-Url: https://codereview.chromium.org/2144823003 Cr-Commit-Position: refs/heads/master@{#407594} [modify] https://crrev.com/84c46fae5808a0a2ac1a851e51208fa9343eaafb/build/android/pylib/local/device/local_device_environment.py [modify] https://crrev.com/84c46fae5808a0a2ac1a851e51208fa9343eaafb/build/android/pylib/local/device/local_device_gtest_run.py [modify] https://crrev.com/84c46fae5808a0a2ac1a851e51208fa9343eaafb/build/android/pylib/local/device/local_device_instrumentation_test_run.py [modify] https://crrev.com/84c46fae5808a0a2ac1a851e51208fa9343eaafb/build/android/pylib/local/device/local_device_perf_test_run.py [modify] https://crrev.com/84c46fae5808a0a2ac1a851e51208fa9343eaafb/build/android/pylib/local/device/local_device_test_run.py
,
Aug 27 2016
|
|||
►
Sign in to add a comment |
|||
Comment 1 by bpastene@chromium.org
, Jul 13 2016