From c6a1efa0280769bf3440a19e25041ef9319c623a Mon Sep 17 00:00:00 2001 From: Sean McGinnis Date: Sat, 15 Feb 2020 14:11:04 -0600 Subject: [PATCH] Install all requirements in docs builds 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 --- tox.ini | 1 + 1 file changed, 1 insertion(+) diff --git a/tox.ini b/tox.ini index ab92c2c1298..c2eb0cbf973 100644 --- a/tox.ini +++ b/tox.ini @@ -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/