Skip to content

Commit bc92f05

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) (cherry picked from commit f66a570)
1 parent 8ff36f1 commit bc92f05

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
@@ -182,6 +182,7 @@ description =
182182
# to install (test-)requirements.txt for docs.
183183
deps =
184184
-c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/victoria}
185+
-r{toxinidir}/requirements.txt
185186
-r{toxinidir}/doc/requirements.txt
186187
commands =
187188
rm -rf doc/build/html doc/build/doctrees

0 commit comments

Comments
 (0)