Merge STDOUT & STDERR of Ansible plays (#704)

Signed-off-by: Christian Berendt <berendt@osism.tech>
osism · Nov 29, 2023 · b00917e · b00917e
1 parent fad312a
commit b00917e
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 6 deletions.
diff --git a/osism/tasks/__init__.py b/osism/tasks/__init__.py
@@ -113,7 +113,7 @@ def run_ansible_in_environment(
                 command,
                 shell=True,
                 stdout=subprocess.PIPE,
-                stderr=subprocess.PIPE,
+                stderr=subprocess.STDOUT,
                 env=env,
             )
         else:
@@ -123,7 +123,7 @@ def run_ansible_in_environment(
                 command,
                 shell=True,
                 stdout=subprocess.PIPE,
-                stderr=subprocess.PIPE,
+                stderr=subprocess.STDOUT,
                 env=env,
             )
 
@@ -138,7 +138,7 @@ def run_ansible_in_environment(
             command,
             shell=True,
             stdout=subprocess.PIPE,
-            stderr=subprocess.PIPE,
+            stderr=subprocess.STDOUT,
             env=env,
         )
 
@@ -154,7 +154,7 @@ def run_ansible_in_environment(
             command,
             shell=True,
             stdout=subprocess.PIPE,
-            stderr=subprocess.PIPE,
+            stderr=subprocess.STDOUT,
         )
 
     # execute local netbox playbooks
@@ -168,7 +168,7 @@ def run_ansible_in_environment(
             command,
             shell=True,
             stdout=subprocess.PIPE,
-            stderr=subprocess.PIPE,
+            stderr=subprocess.STDOUT,
         )
 
     # execute all other roles
@@ -182,7 +182,7 @@ def run_ansible_in_environment(
             command,
             shell=True,
             stdout=subprocess.PIPE,
-            stderr=subprocess.PIPE,
+            stderr=subprocess.STDOUT,
             env=env,
         )
 

diff --git a/releasenotes/notes/merge-stdout-and-stderr-49cc1ea54d16fa4b.yaml b/releasenotes/notes/merge-stdout-and-stderr-49cc1ea54d16fa4b.yaml
@@ -0,0 +1,5 @@
+---
+features:
+  - |
+    STDERR of Ansible runs is now merged with STDOUT. This way the STDERR of Ansible runs
+    is also visible in the STDOUT of the OSISM CLI.