From c6aa7766e1c06be6c3e38480e01b191017bbeec6 Mon Sep 17 00:00:00 2001 From: Sorin Sbarnea Date: Thu, 25 May 2023 19:52:32 +0100 Subject: [PATCH] Disable Ansible debug mode for sub-commands (#284) --- src/ansible_compat/runtime.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/ansible_compat/runtime.py b/src/ansible_compat/runtime.py index c5c798b2..386497ed 100644 --- a/src/ansible_compat/runtime.py +++ b/src/ansible_compat/runtime.py @@ -327,6 +327,10 @@ def run( # ruff: disable=PLR0913 run_func: Callable[..., CompletedProcess] = subprocess_tee.run else: run_func = subprocess.run + env = self.environ if env is None else env.copy() + # Presence of ansible debug variable or config option will prevent us + # from parsing its JSON output due to extra debug messages on stdout. + env["ANSIBLE_DEBUG"] = "0" for _ in range(self.max_retries + 1 if retry else 1): result = run_func( @@ -335,12 +339,12 @@ def run( # ruff: disable=PLR0913 check=False, stdout=subprocess.PIPE, stderr=subprocess.PIPE, - env=env or self.environ, + env=env, cwd=str(cwd) if cwd else None, ) if result.returncode == 0: break - _logger.debug("Environment: %s", env or self.environ) + _logger.debug("Environment: %s", env) if retry: _logger.warning( "Retrying execution failure %s of: %s",