diff options
| author | Michal Nasiadka <mnasiadka@gmail.com> | 2019-11-18 14:04:40 +0000 |
|---|---|---|
| committer | Radosław Piliszek <radoslaw.piliszek@gmail.com> | 2020-05-17 17:02:38 +0200 |
| commit | d8f31e0a5ef28dcc2325fd5951657e211c39a465 (patch) | |
| tree | 6df955916790c96c9144506331ac612ae693d55b /.ansible-lint | |
| parent | fe54fdbc347cac4820e51c4bbd44ad71f6dd4e74 (diff) | |
| download | kolla-ansible-d8f31e0a5ef28dcc2325fd5951657e211c39a465.tar.gz kolla-ansible-d8f31e0a5ef28dcc2325fd5951657e211c39a465.zip | |
CI: Add ansible-lint to tox
* Reworked tox pep8 into linters job, that runs:
- pep8
- bandit
- bashate
- doc8
- yamllint
- ansible-lint (validate-all-files.py + ansible-lint)
* Skip E701 - missing galaxy_info in meta and E602 see [1].
* Skip E301 and E503 - followup later in a separate change
* Added ansible-role-jobs to zuul.d/project.yaml which will run
openstack-tox-linters job in check queue
* Fixed remaining style issue
* Made tox and docs reference the new env for linters
* Dropped pype environment (not supported)
[1]: https://github.com/ansible/ansible-lint/issues/457
Change-Id: I494b4b151804aac8173120e6c6e42bc2fdb00234
Diffstat (limited to '.ansible-lint')
| -rw-r--r-- | .ansible-lint | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/.ansible-lint b/.ansible-lint new file mode 100644 index 000000000..f10b027c8 --- /dev/null +++ b/.ansible-lint @@ -0,0 +1,12 @@ +use_default_rules: true +skip_list: +# [E701] galaxy_info missing in metadata + - '701' +# [E602] https://github.com/ansible/ansible-lint/issues/457 + - '602' +# [E301] Commands should not change things if nothing needs doing +# TODO(mnasiadka): Fix tasks that fail this check in a later iteration + - '301' +# [E503] Tasks that run when changed should likely be handlers +# TODO(mnasiadka): Rework baremetal role to do handlers instead of when: *.changed + - '503'
\ No newline at end of file |