commit 67791bd731abd89126ca56334a43bba75ebeb380
author James E. Blair <jeblair@redhat.com> Wed May 25 13:52:49 2016 -0700
committer James E. Blair <jeblair@redhat.com> Thu May 26 07:19:13 2016 -0700
tree 8db25b229b2684e55976cee7957253462efa0841
parent 45670032a3dd3512fa879b0681917fd0785b5991

Ansible launcher: log stdout/stderr one line at a time

For better grepping.

Change-Id: Idd49bb25c2120d8fa4b8a7df284df4f4c543846d

zuul/launcher/ansiblelaunchserver.py

diff --git a/zuul/launcher/ansiblelaunchserver.py b/zuul/launcher/ansiblelaunchserver.py
index 8c701de..1cb4f98 100644
--- a/zuul/launcher/ansiblelaunchserver.py
+++ b/zuul/launcher/ansiblelaunchserver.py
@@ -846,8 +846,10 @@
             preexec_fn=os.setsid,
         )
         (out, err) = self.ansible_proc.communicate()
-        self.log.debug("Ansible stdout:\n%s" % out)
-        self.log.debug("Ansible stderr:\n%s" % err)
+        for line in out.split('\n'):
+            self.log.debug("Ansible stdout:\n%s" % line)
+        for line in err.split('\n'):
+            self.log.debug("Ansible stderr:\n%s" % line)
         ret = self.ansible_proc.wait()
         self.ansible_proc = None
         return ret == 0
@@ -862,8 +864,10 @@
             preexec_fn=os.setsid,
         )
         (out, err) = proc.communicate()
-        self.log.debug("Ansible post stdout:\n%s" % out)
-        self.log.debug("Ansible post stderr:\n%s" % err)
+        for line in out.split('\n'):
+            self.log.debug("Ansible post stdout:\n%s" % line)
+        for line in err.split('\n'):
+            self.log.debug("Ansible post stderr:\n%s" % line)
         return proc.wait() == 0