PhysicalConsole: Correctly report power_on failure

We've been logging an error, then unconditionally reporting success. Fix
it.

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

diff --git a/teuthology/orchestra/console.py b/teuthology/orchestra/console.py
index c15e02b0fc9b26843d627666294a8b333c7964f4..f2111077463f5cefde99c95dd71d04751700976e 100644 (file)
--- a/teuthology/orchestra/console.py
+++ b/teuthology/orchestra/console.py
@@ -228,9 +228,10 @@ class PhysicalConsole():
                              timeout=self.timeout)
             if r == 0:
                 break
-        if not self.check_power('on'):
+        if self.check_power('on'):
+            log.info('Power on for {s} completed'.format(s=self.shortname))
+        else:
             log.error('Failed to power on {s}'.format(s=self.shortname))
-        log.info('Power on for {s} completed'.format(s=self.shortname))
 
     def power_off(self):
         """