7f156f22c7
This change updates the tox molecule playbook to make it platform agnostic by moving the OS specific package lists into var files that will be dynamically loaded based on the operating system the playbook is being executed against. This is being done so that we can use molecule on OS's like CentOS, Ubuntu, SUSE, Debian, etc. While the only platforms supported at this time are Red Hat based, this change will allow additional var files to be added enabling additional platforms without needing any chanages to our task structure. > The linters job has been updated to exclude vars files from the ansible lint playbook commands. This is done because the playbook commands assume all found files are playbooks. Change-Id: I88f3551838e3676374d0a795631c8769ba40fbf4 Signed-off-by: Kevin Carter <kecarter@redhat.com>
75 lines
2.4 KiB
INI
75 lines
2.4 KiB
INI
[tox]
|
|
minversion = 1.6
|
|
skipsdist = True
|
|
envlist = linters
|
|
|
|
[testenv]
|
|
basepython = python3
|
|
install_command = pip install {opts} {packages}
|
|
deps = -r{toxinidir}/test-requirements.txt
|
|
commands = stestr run {posargs}
|
|
stestr slowest
|
|
|
|
[testenv:py27]
|
|
basepython = python2.7
|
|
|
|
[testenv:bindep]
|
|
# Do not install any requirements. We want this to be fast and work even if
|
|
# system dependencies are missing, since it's used to tell you what system
|
|
# dependencies are missing! This also means that bindep must be installed
|
|
# separately, outside of the requirements files.
|
|
deps = bindep
|
|
commands = bindep test
|
|
|
|
[testenv:docs]
|
|
deps =
|
|
-r{toxinidir}/test-requirements.txt
|
|
-r{toxinidir}/doc/requirements.txt
|
|
commands =
|
|
sphinx-build -E -W -d doc/build/doctrees -b html doc/source/ doc/build/html
|
|
|
|
[testenv:linters]
|
|
passenv =
|
|
# NOTE(pabelanger): if you'd like to run tox -elinters locally, you'll need
|
|
# to export ANSIBLE_ROLES_PATH pointing to the currect repos.
|
|
# see openstack-zuul-jobs-linters job for more information.
|
|
ANSIBLE_ROLES_PATH
|
|
setenv =
|
|
ANSIBLE_LIBRARY= {envsitepackagesdir}/zuul/ansible/base/library
|
|
ANSIBLE_ACTION_PLUGINS = {envsitepackagesdir}/zuul/ansible/base/actiongeneral
|
|
whitelist_externals = bash
|
|
commands =
|
|
flake8 {posargs}
|
|
# Ansible lint
|
|
# [ANSIBLE0012] Commands should not change things if nothing needs doing
|
|
# [204] Lines should be no longer than 160 chars
|
|
bash -c "find playbooks -type f -regex '.*.ya?ml' ! -regex '.*vars\/.*' -print0 | \
|
|
xargs -t -n1 -0 ansible-lint -xANSIBLE0012,204"
|
|
bash -c 'find roles -maxdepth 1 -mindepth 1 -type d -printf "%p/\n" | \
|
|
xargs -t -n1 ansible-lint -xANSIBLE0012,204'
|
|
# Ansible Syntax Check
|
|
bash -c "find playbooks -type f -regex '.*.ya?ml' ! -regex '.*vars\/.*' -exec \
|
|
ansible-playbook --syntax-check -i {toxinidir}/tests/inventory \{\} + > /dev/null"
|
|
{toxinidir}/tools/check_jobs_documented.py
|
|
|
|
[testenv:venv]
|
|
commands = {posargs}
|
|
|
|
[flake8]
|
|
# These are ignored intentionally in openstack-infra projects;
|
|
# please don't submit patches that solely correct them or enable them.
|
|
# E402 - ansible modules put documentation before imports. Align to ansible.
|
|
# W504 - line break after binary operator, we cannot have both
|
|
# W503 and W504 enabled
|
|
ignore = E125,E129,E402,E741,W504,H
|
|
show-source = True
|
|
exclude =
|
|
.venv,
|
|
.tox,
|
|
dist,
|
|
doc,
|
|
build,
|
|
*.egg,
|
|
# vendored files
|
|
roles/use-buildset-registry/module_utils/pytoml.py,
|
|
roles/use-buildset-registry/module_utils/remarshal.py |