Update tox configuration

openstack/openstack-ansible-memcached_server

Code Issues Proposed changes

In order to prepare for implementing requirements management by the
OpenStack requirements management process this patch implements
some changes to the tox configuration:
- The minimum tox version is increased in order to be able to use
 constraints for the python packages.
- The OpenStack upper-constraints are used when preparing the test
 venv for the linters checks.
- Any proxy environment variables set on the test host are passed
 into the venv to enable testing from behind a proxy.
- The environment variables used by Ansible tests are moved into
 a new venv called 'ansible' and this environment is inherited
 by all Ansible-related tests.
- The docs test will clean-up an existing build directory before
 executing the docs build.
- The releasenotes build cannot use upper-constraints at this point,
 so it doesn't.
- The Ansible role download will no longer ignore errors so that any
 problems discovered will result in a failed test.
Change-Id: I2ef319e48a4106d911e2ceaddce2c17094ac1aa7

This commit is contained in:

Jesse Pretorius

2016年06月17日 17:34:57 +01:00

parent 02a1393e68

commit 7b71f3d9c1

1 changed files with 67 additions and 33 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download Patch File Download Diff File Expand all files Collapse all files

100
tox.ini

View File

@@ -1,48 +1,58 @@

[tox]

minversion = 1.6

minversion = 2.0

skipsdist = True

envlist = docs,linters,functional

[testenv]

usedevelop = True

install_command = pip install -U {opts} {packages}

deps = -r{toxinidir}/test-requirements.txt

commands = /usr/bin/find . -type f -name "*.pyc" -delete

install_command =

 pip install -c{env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} {opts} {packages}

deps =

-r{toxinidir}/test-requirements.txt

commands =

/usr/bin/find . -type f -name "*.pyc" -delete

passenv =

HOME

http_proxy

HTTP_PROXY

https_proxy

HTTPS_PROXY

no_proxy

NO_PROXY

whitelist_externals =

bash

git

rm

setenv =

VIRTUAL_ENV={envdir}

ANSIBLE_HOST_KEY_CHECKING = False

ANSIBLE_SSH_CONTROL_PATH = /tmp/%%h-%%r

# TODO (odyssey4me) These are only here as they are non-standard folder

# names for Ansible 1.9.x. We are using the standard folder names for

# Ansible v2.x. We can remove this when we move to Ansible 2.x.

ANSIBLE_ACTION_PLUGINS = {homedir}/.ansible/plugins/action

ANSIBLE_CALLBACK_PLUGINS = {homedir}/.ansible/plugins/callback

ANSIBLE_FILTER_PLUGINS = {homedir}/.ansible/plugins/filter

ANSIBLE_LOOKUP_PLUGINS = {homedir}/.ansible/plugins/lookup

# This is required as the default is the current path or a path specified

# in ansible.cfg

ANSIBLE_LIBRARY = {homedir}/.ansible/plugins/library

# This is required as the default is '/etc/ansible/roles' or a path

# specified in ansible.cfg

ANSIBLE_ROLES_PATH = {homedir}/.ansible/roles:{toxinidir}/..

[testenv:docs]

commands=

bash -c "rm -rf doc/build"

python setup.py build_sphinx

[testenv:releasenotes]

# NOTE(sdague): this target does not use constraints because

# upstream infra does not yet support it. Once that's fixed, we can

# drop the install_command.

install_command =

pip install -U --force-reinstall {opts} {packages}

commands =

sphinx-build -a -E -W -d releasenotes/build/doctrees -b html releasenotes/source releasenotes/build/html

# environment used by the -infra templated docs job

[testenv:venv]

deps = -r{toxinidir}/test-requirements.txt

commands = {posargs}

# NOTE(jaegerandi): this target does not use constraints because

# upstream infra does not yet support it. Once that's fixed, we can

# drop the install_command.

install_command =

pip install -U --force-reinstall {opts} {packages}

commands =

{posargs}

[testenv:pep8]

@@ -57,6 +67,7 @@ commands =

--exclude-dir doc \

{toxinidir} | xargs flake8 --verbose"

[flake8]

# Ignores the following rules due to how ansible modules work in general

# F403 'from ansible.module_utils.basic import *' used;

@@ -82,15 +93,38 @@ commands =

{toxinidir} | xargs bashate --error . --verbose --ignore=E003,E006,E040"

[testenv:ansible-syntax]

[testenv:ansible]

setenv =

{[testenv]setenv}