diff options
| author | Zuul <zuul@review.opendev.org> | 2023-04-14 10:36:45 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2023-04-14 10:36:45 +0000 |
| commit | b142b0ae7af478f6dee5d894e1ad1a5a3f580cf7 (patch) | |
| tree | 0d16c62483c97f88a51e617eabd79bee559374c5 /ansible/action_plugins | |
| parent | be2e353da31c3abf6794385259e1e161502056ef (diff) | |
| parent | 8972241dc9736109f9d98552035e7557c366bf94 (diff) | |
| download | kolla-ansible-b142b0ae7af478f6dee5d894e1ad1a5a3f580cf7.tar.gz kolla-ansible-b142b0ae7af478f6dee5d894e1ad1a5a3f580cf7.zip | |
Merge "Fix merge action plugins verbose output"
Diffstat (limited to 'ansible/action_plugins')
| -rw-r--r-- | ansible/action_plugins/merge_configs.py | 10 | ||||
| -rw-r--r--[-rwxr-xr-x] | ansible/action_plugins/merge_yaml.py | 15 |
2 files changed, 16 insertions, 9 deletions
diff --git a/ansible/action_plugins/merge_configs.py b/ansible/action_plugins/merge_configs.py index 7bbf70654..a82583550 100644 --- a/ansible/action_plugins/merge_configs.py +++ b/ansible/action_plugins/merge_configs.py @@ -171,12 +171,12 @@ class ActionModule(action.ActionBase): 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) @@ -213,7 +213,11 @@ class ActionModule(action.ActionBase): 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 diff --git a/ansible/action_plugins/merge_yaml.py b/ansible/action_plugins/merge_yaml.py index c8a7182bf..648109163 100755..100644 --- a/ansible/action_plugins/merge_yaml.py +++ b/ansible/action_plugins/merge_yaml.py @@ -17,8 +17,7 @@ import os import shutil import tempfile -from yaml import dump -from yaml import safe_load +import yaml from ansible import constants from ansible import errors as ansible_errors @@ -81,7 +80,7 @@ class ActionModule(action.ActionBase): def read_config(self, source): result = None # Only use config if present - if os.access(source, os.R_OK): + if source and os.access(source, os.R_OK): with open(source, 'r') as f: template_data = f.read() @@ -94,7 +93,7 @@ class ActionModule(action.ActionBase): self._templar.environment.loader.searchpath = searchpath template_data = self._templar.template(template_data) - result = safe_load(template_data) + result = yaml.safe_load(template_data) return result or {} def run(self, tmp=None, task_vars=None): @@ -128,7 +127,7 @@ class ActionModule(action.ActionBase): try: result_file = os.path.join(local_tempdir, 'source') with open(result_file, 'w') as f: - f.write(dump(output, default_flow_style=False)) + f.write(yaml.dump(output, default_flow_style=False)) new_task = self._task.copy() new_task.args.pop('sources', None) @@ -147,7 +146,11 @@ class ActionModule(action.ActionBase): 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 |
