[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
)
This commit is contained in:
parent
b6c2bb362f
commit
49f50c9f35
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/xena}
|
||||
-r{toxinidir}/requirements.txt
|
||||
-r{toxinidir}/doc/requirements.txt
|
||||
commands =
|
||||
rm -rf doc/build
|
||||
|
|
Loading…
Reference in New Issue