From: Zack Cerza <zack@redhat.com>
Date: Thu, 18 Aug 2016 23:12:22 +0000 (-0600)
Subject: PhysicalConsole._pexpect_spawn(): pass logfile arg
X-Git-Tag: 1.1.0~544^2~6
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=29d1b9638abbc97936a41ed6b58dadd6e7ecff3f;p=teuthology.git

PhysicalConsole._pexpect_spawn(): pass logfile arg

... directly when calling pexpect.spawn()

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

diff --git a/teuthology/orchestra/console.py b/teuthology/orchestra/console.py
index 9f9b88ffa6..7ccd03e275 100644
--- a/teuthology/orchestra/console.py
+++ b/teuthology/orchestra/console.py
@@ -38,9 +38,10 @@ class PhysicalConsole():
             log.error('Must set ipmi_user, ipmi_password, and ipmi_domain in .teuthology.yaml')  # noqa
         full_command = self._build_command(cmd)
         log.debug('pexpect command: %s', full_command)
-        child = pexpect.spawn(full_command)
-        if self.logfile:
-            child.logfile = self.logfile
+        child = pexpect.spawn(
+            full_command,
+            logfile=self.logfile,
+        )
         return child
 
     def _build_command(self, subcommand):