diff options
| author | eseidel@chromium.org <eseidel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-17 21:51:21 +0000 |
|---|---|---|
| committer | eseidel@chromium.org <eseidel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-17 21:51:21 +0000 |
| commit | 7b54a872367bf6f09bca4ff7c810cfcdd729b539 (patch) | |
| tree | bcc7a0927b1fef9a9079879c996f81648e9d5b1c /build/android/buildbot | |
| parent | 4a6274b2228053c6b87d72abc559db1fd6ba4245 (diff) | |
| download | chromium_src-7b54a872367bf6f09bca4ff7c810cfcdd729b539.zip chromium_src-7b54a872367bf6f09bca4ff7c810cfcdd729b539.tar.gz chromium_src-7b54a872367bf6f09bca4ff7c810cfcdd729b539.tar.bz2 | |
Make provision_devices failures abort the build
Otherwise if the devices are gone lots of tests will just fail.
Already fixed this for recipe based builders in crbug.com/393681
I don't know how to test this, but happy to write one if shown.
BUG=393681
NOTRY=True
Review URL: https://codereview.chromium.org/390993003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283890 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'build/android/buildbot')
| -rwxr-xr-x | build/android/buildbot/bb_device_steps.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/build/android/buildbot/bb_device_steps.py b/build/android/buildbot/bb_device_steps.py index de70d80..1d0ba35 100755 --- a/build/android/buildbot/bb_device_steps.py +++ b/build/android/buildbot/bb_device_steps.py @@ -414,7 +414,7 @@ def ProvisionDevices(options): provision_cmd.append('--auto-reconnect') if options.skip_wipe: provision_cmd.append('--skip-wipe') - RunCmd(provision_cmd) + RunCmd(provision_cmd, halt_on_failure=True) def DeviceStatusCheck(options): |