From da44a7f0cbc593561af2dd7950f090748ac6436d Mon Sep 17 00:00:00 2001 From: Andrew Smith Date: Thu, 28 Nov 2013 10:53:00 +1000 Subject: [PATCH] I think that resultant is meant to be the contents of the combined file MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit It is used for the transfer and as a diff param but isn’t actually defined anywhere before it’s used. This seemed like the least bad place to set it. --- lib/ansible/runner/action_plugins/assemble.py | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/ansible/runner/action_plugins/assemble.py b/lib/ansible/runner/action_plugins/assemble.py index dbcad0cc9a..d20b68b1d5 100644 --- a/lib/ansible/runner/action_plugins/assemble.py +++ b/lib/ansible/runner/action_plugins/assemble.py @@ -71,6 +71,7 @@ class ActionModule(object): remote_md5 = self.runner._remote_md5(conn, tmp, dest) if pathmd5 != remote_md5: + resultant = file(path).read() if self.runner.diff: dest_result = self.runner._execute_module(conn, tmp, 'slurp', "path=%s" % dest, inject=inject, persist_files=True) if 'content' in dest_result.result: