7 Commits

Author SHA1 Message Date
Jenkins
c327263467 Merge "Rework tox -e linters" 2017-07-20 05:14:24 +00:00
Paul Belanger
f40a7fbe1e
Install zuul to access zuul_console
When we lint our playbooks, we need to also have zuul install our
custom action plugins.

Also default to python3 so we can properly install zuul into tox.

Change-Id: Ib7e39e43005aa73c9d482af6becef3408d097e15
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
2017-07-19 13:06:30 -04:00
Paul Belanger
74d237a3f7
Rework tox -e linters
Move our syntax-check to be first, logic being if we have bad syntax
and ansible-lint will most likely fail.

Clean our redirects to only output failures. It would be nice is
ansible-playbook / ansible-lint could use the same redirects, but
sadly they cannot.

Change-Id: Ia3c5966d60d28b81e2f2c6a446116bb37df6ab23
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
2017-07-18 10:48:54 -04:00
Paul Belanger
8826f3afd9
Rename pep8 to linters for tox
Using linters makes more sense, for playbooks / roles. Make the
switch.

Change-Id: I9694f95c3c041f9816c83d60680bfde50e1fb858
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
2017-07-17 16:54:29 -04:00
Paul Belanger
8e812f1eb4
Include ansible-playbook syntax-check for tox pep8
Syntax check our playbooks / roles when we run pep8. This way we can
quickly tests if our playbooks are valid.

Change-Id: Id9b44216820a2b0702e1cfa46cd7603c89ed40d3
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
2017-07-17 16:53:55 -04:00
Paul Belanger
829297c37a
Ensure we load roles for linting
Did didn't have ansible-lint setup properly, as a results our roles
weren't actually linted properly.

Fix variable linting issues and ignore ANSIBLE0012.

Change-Id: I07aa940245e700c9f08df0f1920720f0ed9d3de0
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
2017-07-17 16:50:06 -04:00
James E. Blair
e5ebc400cb Add initial license, docs, and other config
Change-Id: Ie016aa9a3494bea1e37b7eee4a970d5772317d47
2017-06-08 14:01:21 -07:00