From 9556bcb9c2c21f1d3ee793a85e86c4bb899f6550 Mon Sep 17 00:00:00 2001 From: Bert Van Vreckem Date: Tue, 8 Dec 2015 12:55:09 +0100 Subject: [PATCH] Also show stdout when validation fails. This fixes #2498 --- files/copy.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/files/copy.py b/files/copy.py index 5dd1e9935e6..1e187306e46 100644 --- a/files/copy.py +++ b/files/copy.py @@ -279,7 +279,7 @@ def main(): # os.path.exists() can return false in some # circumstances where the directory does not have # the execute bit for the current user set, in - # which case the stat() call will raise an OSError + # which case the stat() call will raise an OSError os.stat(os.path.dirname(dest)) except OSError, e: if "permission denied" in str(e).lower(): @@ -308,7 +308,7 @@ def main(): module.fail_json(msg="validate must contain %%s: %s" % (validate)) (rc,out,err) = module.run_command(validate % src) if rc != 0: - module.fail_json(msg="failed to validate: rc:%s error:%s" % (rc,err)) + module.fail_json(msg="failed to validate", exit_status=rc, stdout=out, stderr=err) if remote_src: _, tmpdest = tempfile.mkstemp(dir=os.path.dirname(dest)) shutil.copy2(src, tmpdest)