From d8f31e0a5ef28dcc2325fd5951657e211c39a465 Mon Sep 17 00:00:00 2001 From: Michal Nasiadka Date: Mon, 18 Nov 2019 14:04:40 +0000 Subject: 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 --- .ansible-lint | 12 ++++++++++++ 1 file changed, 12 insertions(+) create mode 100644 .ansible-lint (limited to '.ansible-lint') 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 -- cgit v1.2.3