openstack/cinder
Code Issues Proposed changes

Install all requirements in docs builds

Browse Source 
Change If86dd619402495d9d4470b14cb270fcf53db6794 removed installing
requirements from requirements.txt for doc jobs. These are actually
needed since the local project gets installed into the venv in order to
generate docs from the included docstrings. That installation works, but
since the requirements were not installed explicitly via the deps, those
requirements end up being installed without the upper-constraints
enforcement. This is especially problematic on stable branches, where
cinder install ends up pulling in requirements newer than what was used
for that branch.

Change-Id: Icdf80000287a43e1ff355d976f5a607543fd6d20
Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
This commit is contained in:
Sean McGinnis 2020-02-15 14:11:04 -06:00
parent 3a73123472
commit c6a1efa028
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
tox.ini
View File

@ -127,6 +127,7 @@ commands = {posargs}
[testenv:docs]
deps =
-c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}
-r{toxinidir}/requirements.txt
-r{toxinidir}/doc/requirements.txt
commands =
rm -fr doc/source/contributor/api/