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

Issue 495123002: [Android] Fix provisioning for perf bots on L and/or user builds. (Closed)

Created:
6 years, 4 months ago by jbudorick
Modified:
6 years, 4 months ago
Reviewers:
navabi1, tonyg
CC:
chromium-reviews, klundberg+watch_chromium.org, yfriedman+watch_chromium.org, ilevy-cc_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

[Android] Fix provisioning for perf bots on L and/or user builds. BUG=405950 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=291231

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -3 lines) Patch
M build/android/provision_devices.py View 3 chunks +8 lines, -3 lines 2 comments Download

Messages

Total messages: 6 (0 generated)
jbudorick
Fixing a couple of issues from the parallel provisioning CL: - Reboots on L are ...
6 years, 4 months ago (2014-08-21 17:09:34 UTC) #1
tonyg
https://codereview.chromium.org/495123002/diff/1/build/android/provision_devices.py File build/android/provision_devices.py (right): https://codereview.chromium.org/495123002/diff/1/build/android/provision_devices.py#newcode154 build/android/provision_devices.py:154: device.EnableRoot() I think the old code would skip the ...
6 years, 4 months ago (2014-08-21 17:12:58 UTC) #2
jbudorick
https://codereview.chromium.org/495123002/diff/1/build/android/provision_devices.py File build/android/provision_devices.py (right): https://codereview.chromium.org/495123002/diff/1/build/android/provision_devices.py#newcode141 build/android/provision_devices.py:141: device.EnableRoot() @tonyg: it'll skip the wipe here. if we ...
6 years, 4 months ago (2014-08-21 17:15:15 UTC) #3
jbudorick
The CQ bit was checked by jbudorick@chromium.org
6 years, 4 months ago (2014-08-21 17:15:19 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jbudorick@chromium.org/495123002/1
6 years, 4 months ago (2014-08-21 17:16:40 UTC) #5
commit-bot: I haz the power
6 years, 4 months ago (2014-08-21 23:15:59 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (1) as 291231

Powered by Google App Engine
This is Rietveld 408576698