External-Mirrors/awx
2
0
Fork 0
mirror of https://github.com/ansible/awx.git synced 2026-01-14 03:10:42 -03:30
Code Issues Packages Projects Releases Wiki Activity

Fixes AC-346. Allow playbooks with invalid YAML syntax to show up in list of available playbooks.

Browse Source
This commit is contained in:
Chris Church 2013-08-28 09:58:36 -04:00
parent 8d16485f7f
commit 92e0beb5cb
2 changed files with 13 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
awx/main/models/__init__.py
View File

@ -6,6 +6,7 @@ import hmac
import json
import logging
import os
import re
import shlex
# PyYAML
@ -669,6 +670,7 @@ class Project(CommonModel):
@property
def playbooks(self):
valid_re = re.compile(r'^\s*?-?\s*?(?:hosts|include):\s*?.*?$')
results = []
project_path = self.get_project_path()
if project_path:
@ -677,17 +679,17 @@ class Project(CommonModel):
if os.path.splitext(filename)[-1] != '.yml':
continue
playbook = os.path.join(dirpath, filename)
# Filter any invalid YAML files.
try:
data = yaml.safe_load(file(playbook).read())
except (IOError, yaml.YAMLError):
continue
# Filter files that do not have either hosts or top-level
# includes.
# includes. Use regex to allow files with invalid YAML to
# show up.
matched = False
try:
if 'hosts' not in data[0] and 'include' not in data[0]:
continue
except (TypeError, IndexError, KeyError):
for line in file(playbook):
if valid_re.match(line):
matched = True
except IOError:
continue
if not matched:
continue
playbook = os.path.relpath(playbook, project_path)
# Filter files in a roles subdirectory.

4
awx/main/tests/projects.py
View File

@ -118,11 +118,11 @@ class ProjectsTest(BaseTest):
self.assertEqual(len(project.playbooks), 1)
write_test_file(project, 'blah.yml', '')
self.assertEqual(len(project.playbooks), 1)
# Invalid YAML
# Invalid YAML (now allowed to show)
project = self.projects[4]
self.assertEqual(len(project.playbooks), 1)
write_test_file(project, 'blah.yml', TEST_PLAYBOOK + '----')
self.assertEqual(len(project.playbooks), 1)
self.assertEqual(len(project.playbooks), 2)
# No hosts or includes
project = self.projects[5]
self.assertEqual(len(project.playbooks), 1)