From patchwork Tue Jan 3 18:29:13 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Paul Larson X-Patchwork-Id: 6033 Return-Path: X-Original-To: patchwork@peony.canonical.com Delivered-To: patchwork@peony.canonical.com Received: from fiordland.canonical.com (fiordland.canonical.com [91.189.94.145]) by peony.canonical.com (Postfix) with ESMTP id 6FDFF23E14 for ; Tue, 3 Jan 2012 18:29:16 +0000 (UTC) Received: from mail-ee0-f52.google.com (mail-ee0-f52.google.com [74.125.83.52]) by fiordland.canonical.com (Postfix) with ESMTP id 52E83A185A5 for ; Tue, 3 Jan 2012 18:29:16 +0000 (UTC) Received: by eeke52 with SMTP id e52so17033605eek.11 for ; Tue, 03 Jan 2012 10:29:16 -0800 (PST) Received: by 10.205.132.148 with SMTP id hu20mr9624018bkc.96.1325615355957; Tue, 03 Jan 2012 10:29:15 -0800 (PST) X-Forwarded-To: linaro-patchwork@canonical.com X-Forwarded-For: patch@linaro.org linaro-patchwork@canonical.com Delivered-To: patches@linaro.org Received: by 10.205.82.144 with SMTP id ac16cs347259bkc; Tue, 3 Jan 2012 10:29:15 -0800 (PST) Received: by 10.180.81.72 with SMTP id y8mr116327301wix.14.1325615353759; Tue, 03 Jan 2012 10:29:13 -0800 (PST) Received: from indium.canonical.com (indium.canonical.com. [91.189.90.7]) by mx.google.com with ESMTPS id k42si23215096weq.41.2012.01.03.10.29.13 (version=TLSv1/SSLv3 cipher=OTHER); Tue, 03 Jan 2012 10:29:13 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of bounces@canonical.com designates 91.189.90.7 as permitted sender) client-ip=91.189.90.7; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of bounces@canonical.com designates 91.189.90.7 as permitted sender) smtp.mail=bounces@canonical.com Received: from ackee.canonical.com ([91.189.89.26]) by indium.canonical.com with esmtp (Exim 4.71 #1 (Debian)) id 1Ri96n-0003Nv-FW for ; Tue, 03 Jan 2012 18:29:13 +0000 Received: from ackee.canonical.com (localhost [127.0.0.1]) by ackee.canonical.com (Postfix) with ESMTP id 6912AE0785 for ; Tue, 3 Jan 2012 18:29:13 +0000 (UTC) MIME-Version: 1.0 X-Launchpad-Project: lava-dispatcher X-Launchpad-Branch: ~linaro-validation/lava-dispatcher/trunk X-Launchpad-Message-Rationale: Subscriber X-Launchpad-Branch-Revision-Number: 186 X-Launchpad-Notification-Type: branch-revision To: Linaro Patch Tracker From: noreply@launchpad.net Subject: [Branch ~linaro-validation/lava-dispatcher/trunk] Rev 186: Fix detection of lava-test install failure (lp: 905457) Message-Id: <20120103182913.14049.26324.launchpad@ackee.canonical.com> Date: Tue, 03 Jan 2012 18:29:13 -0000 Reply-To: noreply@launchpad.net Sender: bounces@canonical.com Errors-To: bounces@canonical.com Precedence: bulk X-Generated-By: Launchpad (canonical.com); Revision="14616"; Instance="launchpad-lazr.conf" X-Launchpad-Hash: 6dec2052e0eef92ed3f93d876f0be68cd530ea47 Merge authors: Spring Zhang (qzhang) Related merge proposals: https://code.launchpad.net/~qzhang/lava-dispatcher/fix-905457/+merge/86500 proposed by: Spring Zhang (qzhang) review: Approve - Paul Larson (pwlars) ------------------------------------------------------------ revno: 186 [merge] committer: Paul Larson branch nick: lava-dispatcher timestamp: Tue 2012-01-03 12:25:33 -0600 message: Fix detection of lava-test install failure (lp: 905457) modified: lava_dispatcher/actions/lava-test.py --- lp:lava-dispatcher https://code.launchpad.net/~linaro-validation/lava-dispatcher/trunk You are subscribed to branch lp:lava-dispatcher. To unsubscribe from this branch go to https://code.launchpad.net/~linaro-validation/lava-dispatcher/trunk/+edit-subscription === modified file 'lava_dispatcher/actions/lava-test.py' --- lava_dispatcher/actions/lava-test.py 2011-12-15 00:54:40 +0000 +++ lava_dispatcher/actions/lava-test.py 2011-12-21 06:21:39 +0000 @@ -44,11 +44,13 @@ #Test if lava-test installed try: - session.run('lava-test help', response="list-test", timeout=60) + rc = session.run('lava-test help', response="list-test", timeout=60) except: tb = traceback.format_exc() client.sio.write(tb) raise CriticalError("lava-test deployment failed") + if rc != 0: + raise CriticalError("lava-test deployment failed") class cmd_lava_test_run(BaseAction):