From: Zack Cerza <zack@redhat.com>
Date: Tue, 5 Dec 2017 21:24:10 +0000 (-0700)
Subject: PhysicalConsole: Correctly report power_off failure
X-Git-Tag: 1.1.0~372^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F1134%2Fhead;p=teuthology.git

PhysicalConsole: Correctly report power_off failure

Signed-off-by: Zack Cerza <zack@redhat.com>
---

diff --git a/teuthology/orchestra/console.py b/teuthology/orchestra/console.py
index f21110774..82dd8e687 100644
--- a/teuthology/orchestra/console.py
+++ b/teuthology/orchestra/console.py
@@ -245,9 +245,10 @@ class PhysicalConsole():
                              timeout=self.timeout)
             if r == 0:
                 break
-        if not self.check_power('off', 60):
+        if self.check_power('off', 60):
+            log.info('Power off for {s} completed'.format(s=self.shortname))
+        else:
             log.error('Failed to power off {s}'.format(s=self.shortname))
-        log.info('Power off for {s} completed'.format(s=self.shortname))
 
     def power_off_for_interval(self, interval=30):
         """