Skip to content

Commit 1ac0d69

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)
1 parent 3b4b609 commit 1ac0d69

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
@@ -228,6 +228,7 @@ description =
228228
# to install (test-)requirements.txt for docs.
229229
deps =
230230
-c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/yoga}
231+
-r{toxinidir}/requirements.txt
231232
-r{toxinidir}/doc/requirements.txt
232233
extras =
233234
commands =

0 commit comments

Comments
 (0)