commit f70650c57a8f44f80055f5d136c4808a79d67c4d
author Chris Masone <cmasone@chromium.org> Wed May 16 08:52:12 2012 -0700
committer Gerrit <chrome-bot@google.com> Thu May 17 07:43:53 2012 -0700
tree 330760038dd369b617acbba1c0979b63dfa2fd81
parent 3e46a418603c1ed6f9d297085460a9d28ffac914

[autotest] Re-land "Put dev server error strings in exceptions"

We've been ignoring error messages coming back from the dev server.
Strip out the HTML that comes in (some of) them and wrap them
in exceptions.

This reverts commit 0dcf3a4c80b66d7598f3d7391bc1f9bf5492d8e2.

BUG=chromium-os:30279
TEST=unit
TEST=use atest suite create to run a suite that doesn't exist on build that doesn't exist.

Change-Id: I9d684d8b1c959d17c4b17082ef6892112b860493
Reviewed-on: https://gerrit.chromium.org/gerrit/22847
Reviewed-by: Scott Zawalski <scottz@chromium.org>
Tested-by: Chris Masone <cmasone@chromium.org>
Commit-Ready: Chris Masone <cmasone@chromium.org>

frontend/afe/site_rpc_interface.py

diff --git a/frontend/afe/site_rpc_interface.py b/frontend/afe/site_rpc_interface.py
index 818f3d1..f82731b 100644
--- a/frontend/afe/site_rpc_interface.py
+++ b/frontend/afe/site_rpc_interface.py
@@ -71,8 +71,10 @@
     ds = dev_server.DevServer.create()
     timings['download_started_time'] = datetime.datetime.now().strftime(
         time_fmt)
-    if not ds.trigger_download(build, synchronous=False):
-        raise error.StageBuildFailure("Server error while staging " + build)
+    try:
+        ds.trigger_download(build, synchronous=False)
+    except dev_server.DevServerException as e:
+        raise error.StageBuildFailure(e)
     timings['payload_finished_time'] = datetime.datetime.now().strftime(
         time_fmt)