ansible/community.general
1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2024-09-14 20:13:21 +02:00
Code Releases Activity

better error on bad playbook encoding

Browse source
This commit is contained in:
Brian Coca 2017-09-14 10:15:45 -04:00 committed by Toshio Kuratomi
parent dfc0084f01
commit 1950bcc14e
1 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
lib/ansible/playbook/__init__.py
View file

@ -23,7 +23,7 @@ import os
from ansible import constants as C from ansible import constants as C
from ansible.errors import AnsibleParserError from ansible.errors import AnsibleParserError
from ansible.module_utils._text import to_text from ansible.module_utils._text import to_text, to_native
from ansible.playbook.play import Play from ansible.playbook.play import Play
from ansible.playbook.playbook_include import PlaybookInclude from ansible.playbook.playbook_include import PlaybookInclude
from ansible.plugins.loader import get_all_plugin_loaders from ansible.plugins.loader import get_all_plugin_loaders
@ -74,7 +74,11 @@ class Playbook:
if os.path.isdir(plugin_path): if os.path.isdir(plugin_path):
obj.add_directory(plugin_path) obj.add_directory(plugin_path)
ds = self._loader.load_from_file(os.path.basename(file_name)) try:
ds = self._loader.load_from_file(os.path.basename(file_name))
except UnicodeDecodeError as e:
raise AnsibleParserError("Could not read playbook (%s) due to encoding issues: %s" % (file_name, to_native(e)))
if not isinstance(ds, list): if not isinstance(ds, list):
# restore the basedir in case this error is caught and handled # restore the basedir in case this error is caught and handled
self._loader.set_basedir(cur_basedir) self._loader.set_basedir(cur_basedir)