Skip to content

Commit 64cc084

Browse files
author
Elod Illes
committed
[CI] Install dependencies for docs target
When tox 'docs' target is called, first it installs the dependencies (listed in 'deps') in 'installdeps' phase, then it installs nova (with its requirements) in 'develop-inst' phase. In the latter case 'deps' is not used so that the constraints defined in 'deps' are not used. This could lead to failures on stable branches when new packages are released that break the build. To avoid this, the simplest solution is to pre-install requirements, i.e. add requirements.txt to 'docs' tox target. Conflicts: tox.ini NOTE(elod.illes): conflict is due to branch specific upper constraints file link. Change-Id: I4471d4488d336d5af0c23028724c4ce79d6a2031 (cherry picked from commit 494e8d7) (cherry picked from commit 1ac0d69)
1 parent 8348484 commit 64cc084

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

tox.ini

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -190,6 +190,7 @@ description =
190190
# to install (test-)requirements.txt for docs.
191191
deps =
192192
-c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/xena}
193+
-r{toxinidir}/requirements.txt
193194
-r{toxinidir}/doc/requirements.txt
194195
commands =
195196
rm -rf doc/build/html doc/build/doctrees

0 commit comments

Comments
 (0)