Skip to content

Commit f66a570

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) (cherry picked from commit 64cc084)
1 parent 327693a commit f66a570

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/wallaby}
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)