From 5ba41a9575d300738520a10a4caf54e76302bc50 Mon Sep 17 00:00:00 2001 From: Toshio Kuratomi Date: Fri, 10 Aug 2018 10:27:12 -0700 Subject: [PATCH] Remove call to parse_returned_data when removing tmpdir _parsed_return_data should only be used with the return from a module. This location was invoking a remote shell command rather than a module so we don't want it here. --- lib/ansible/plugins/action/__init__.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/lib/ansible/plugins/action/__init__.py b/lib/ansible/plugins/action/__init__.py index 162c5441184..9f55f299aa8 100644 --- a/lib/ansible/plugins/action/__init__.py +++ b/lib/ansible/plugins/action/__init__.py @@ -341,8 +341,7 @@ class ActionBase(with_metaclass(ABCMeta, object)): # If ssh breaks we could leave tmp directories out on the remote system. tmp_rm_res = self._low_level_execute_command(cmd, sudoable=False) - tmp_rm_data = self._parse_returned_data(tmp_rm_res) - if tmp_rm_data.get('rc', 0) != 0: + if tmp_rm_res.get('rc', 0) != 0: display.warning('Error deleting remote temporary files (rc: %s, stderr: %s})' % (tmp_rm_res.get('rc'), tmp_rm_res.get('stderr', 'No error string available.'))) else: