Custom ansible playbooks checker using linter
Added custom style rules checker that checks whether the playbooks follow YAML dictionary format or not. Guidelines for writing Ansible playbooks is located here: http://docs.openstack.org/developer/openstack-ansible/ developer-docs/contribute.html#ansible-style-guide Change-Id: I9ad0d90017a0ce38f97dcbd43cbe8c1c13f185ed Closes-Bug: 1629033
This commit is contained in:
parent
fdcefabecc
commit
7a576b2e8b
35
ansible-lint/YAMLdictchecker.py
Executable file
35
ansible-lint/YAMLdictchecker.py
Executable file
@ -0,0 +1,35 @@
|
|||||||
|
#!/usr/bin/env python
|
||||||
|
# Copyright 2016, Rackspace US, Inc.
|
||||||
|
#
|
||||||
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
||||||
|
# you may not use this file except in compliance with the License.
|
||||||
|
# You may obtain a copy of the License at
|
||||||
|
#
|
||||||
|
# http://www.apache.org/licenses/LICENSE-2.0
|
||||||
|
#
|
||||||
|
# Unless required by applicable law or agreed to in writing, software
|
||||||
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
||||||
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||||
|
# See the License for the specific language governing permissions and
|
||||||
|
# limitations under the License.
|
||||||
|
#
|
||||||
|
|
||||||
|
|
||||||
|
from ansiblelint import AnsibleLintRule
|
||||||
|
|
||||||
|
|
||||||
|
class YAMLdictchecker(AnsibleLintRule):
|
||||||
|
id = 'OSA0001'
|
||||||
|
shortdesc = 'Please use ":" YAML dictionary format instead of "="'
|
||||||
|
description = 'Pleae follow YAML dicitionary format while creating'
|
||||||
|
'task and other roles in Ansible'
|
||||||
|
'Follow this url for examples of how to use YAML dictionary '
|
||||||
|
'format. "http://docs.openstack.org/developer/openstack-ansible/'
|
||||||
|
'developer-docs/contribute.html#ansible-style-guide"'
|
||||||
|
tags = ['task']
|
||||||
|
|
||||||
|
def match(self, file, line):
|
||||||
|
for l in line.split(" "):
|
||||||
|
if "=" in l:
|
||||||
|
return True
|
||||||
|
return False
|
@ -37,4 +37,4 @@ export COMMON_TESTS_PATH="${WORKING_DIR}/tests/common"
|
|||||||
source "${COMMON_TESTS_PATH}/test-ansible-env-prep.sh"
|
source "${COMMON_TESTS_PATH}/test-ansible-env-prep.sh"
|
||||||
|
|
||||||
# Execute ansible-lint
|
# Execute ansible-lint
|
||||||
ansible-lint ${WORKING_DIR}
|
ansible-lint ${WORKING_DIR} -R -r ${WORKING_DIR}/ansible-lint/
|
||||||
|
Loading…
x
Reference in New Issue
Block a user