[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 manila (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: I241ab2788b870060e1956a8fc029705378a2a8f4 (cherry picked from commit9b4c8c9251
) (cherry picked from commit6b644fa8ca
) (cherry picked from commit49f50c9f35
)
This commit is contained in:
parent
f5b2cdc23d
commit
7102626e14
4
tox.ini
4
tox.ini
|
@ -67,8 +67,12 @@ commands = oslopolicy-sample-generator --config-file=etc/manila/manila-policy-ge
|
|||
commands = {posargs}
|
||||
|
||||
[testenv:docs]
|
||||
# NOTE(elod.illes): requirements.txt is needed because otherwise
|
||||
# dependencies are installed during 'develop-inst' tox phase without
|
||||
# constraints which could cause failures in stable branches.
|
||||
deps =
|
||||
-c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/wallaby}
|
||||
-r{toxinidir}/requirements.txt
|
||||
-r{toxinidir}/doc/requirements.txt
|
||||
commands =
|
||||
rm -rf doc/build
|
||||
|
|
Loading…
Reference in New Issue