Skip to content

Commit

Permalink
Fix merge action plugins verbose output
Browse files Browse the repository at this point in the history
This change fixes the output 'module_args' information of the plugins
'merge_configs' and 'merge_yaml' when Ansible is executed in maximum
verbose mode. Now all the plugin options are displayed instead of
standard 'copy' plugin options only.

Change-Id: Ie2d9a0501fe29bfd854eb31258f282b197855948
Signed-off-by: Maksim Malchuk <[email protected]>
  • Loading branch information
mmalchuk committed Apr 1, 2023
1 parent 137e64b commit 46000d4
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 4 deletions.
10 changes: 7 additions & 3 deletions kayobe/plugins/action/merge_configs.py
Original file line number Diff line number Diff line change
Expand Up @@ -175,12 +175,12 @@ def run(self, tmp=None, task_vars=None):
del tmp # not used

sources = self._task.args.get('sources', None)
whitespace = self._task.args.get('whitespace', True)

if not isinstance(sources, list):
sources = [sources]

config = OverrideConfigParser(
whitespace=self._task.args.get('whitespace', True))
config = OverrideConfigParser(whitespace=whitespace)

for source in sources:
self.read_config(source, config)
Expand Down Expand Up @@ -217,7 +217,11 @@ def run(self, tmp=None, task_vars=None):
loader=self._loader,
templar=self._templar,
shared_loader_obj=self._shared_loader_obj)
result.update(copy_action.run(task_vars=task_vars))
copy_result = copy_action.run(task_vars=task_vars)
copy_result['invocation']['module_args'].update({
'src': result_file, 'sources': sources,
'whitespace': whitespace})
result.update(copy_result)
finally:
shutil.rmtree(local_tempdir)
return result
6 changes: 5 additions & 1 deletion kayobe/plugins/action/merge_yaml.py
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,11 @@ def run(self, tmp=None, task_vars=None):
loader=self._loader,
templar=self._templar,
shared_loader_obj=self._shared_loader_obj)
result.update(copy_action.run(task_vars=task_vars))
copy_result = copy_action.run(task_vars=task_vars)
copy_result['invocation']['module_args'].update({
'src': result_file, 'sources': sources,
'extend_lists': extend_lists})
result.update(copy_result)
finally:
shutil.rmtree(local_tempdir)
return result
Expand Down

0 comments on commit 46000d4

Please sign in to comment.