diff options
| author | Radosław Piliszek <radoslaw.piliszek@gmail.com> | 2022-09-29 16:37:39 +0200 |
|---|---|---|
| committer | Radosław Piliszek <radoslaw.piliszek@gmail.com> | 2022-09-29 16:37:39 +0200 |
| commit | dc9ab52d0d7c441d082852fe5123741463469731 (patch) | |
| tree | 61e66049abcc1f659ef122faa9a6b408dc7293dc /tox.ini | |
| parent | 4c60cc524ca2b11dae6a0527c82ec5760c894e91 (diff) | |
| download | kolla-ansible-dc9ab52d0d7c441d082852fe5123741463469731.tar.gz kolla-ansible-dc9ab52d0d7c441d082852fe5123741463469731.zip | |
[tox] Fix installation of requirements
Both venv and linters (and its children) environments install
kolla-ansible and thus also install the requirements.
However, they were doing this post-factum and thus without the
constraints pin.
This patch also removes the installation of test-requirements
in venv as it is meant to be used for running the software and
we already have environments for unit tests.
The doc requirements are left in place because docs mention
that ``tox -e venv -- reno`` should work. They should be harmless
but I am open to removing them as well.
Change-Id: I15f1ecc216c9ba81dad740c372d297adf279a945
Diffstat (limited to 'tox.ini')
| -rw-r--r-- | tox.ini | 3 |
1 files changed, 2 insertions, 1 deletions
@@ -41,7 +41,7 @@ commands = [testenv:venv] deps = -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master} - -r{toxinidir}/test-requirements.txt + -r{toxinidir}/requirements.txt -r{toxinidir}/doc/requirements.txt commands = {posargs} @@ -93,6 +93,7 @@ setenv = ANSIBLE_FILTER_PLUGINS = {toxinidir}/ansible/filter_plugins deps = -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master} + -r{toxinidir}/requirements.txt -r{toxinidir}/lint-requirements.txt allowlist_externals = bash find |