Go to file

Colleen Murphy d1956bd9f7 Fix ansible-lint job

The 'linters' tox environment was not running on roles, it was only
running on playbooks. This change adds a command to the linters
environment to ensure all roles are linted.

Since these weren't being linted, there were some problems with them.
The first was a warning about usage of the shell module. Both of these
usages seemed appropriate, so this patch adds a skip_ansible_lint tag to
each task that was failing. The second is a warning "no action detected
in task" for zuul modules. This is due to the fact that the linter
cannot find the custom module. One option is to set the ANSIBLE_LIBRARY
path to point to the zuul ansible library directory[1], but the linter
virtualenv does not actually have zuul installed. Instead, we just
disable the linter for the failing tasks.

This also cleans up a comment in the tox file that was referring to a
nonexistent zuul job.

[1] https://github.com/willthames/ansible-review/issues/16

Change-Id: Ie49da9a09733b623bb25c5a4c8aa07eacacf4b33

2018-03-28 23:10:22 +02:00

doc/source

Fix pep8/yamllint errors

2018-01-02 06:43:26 +00:00

playbooks

Revert "Revert "Deploy the build ssh key to auth_keys""

2018-03-06 18:51:30 +00:00

roles

Fix ansible-lint job

2018-03-28 23:10:22 +02:00

tests

Include ansible-playbook syntax-check for tox pep8