From f23379938e89409c419272e32bab43e1924b058d Mon Sep 17 00:00:00 2001
From: yatinkarel <ykarel@redhat.com>
Date: Thu, 16 Feb 2023 13:46:25 +0530
Subject: [PATCH] Use upper-constraints in docs job

Since we use usedevelop=True, as a result neutron-fwaas
installation happens at 'develop-inst' phase and it installs
requirements from requirements.txt without considering
tox constraints, This can lead to issue where broken
requirements are installed in docs job and as a result
it will fail.

In order to get constraints be considered for this case, add
requirements.txt to deps for docs env(same as [1]).

[1 https://review.opendev.org/q/Ic8f6472fe708d12f8fc129729465367083e1095a

Change-Id: Iea9556f6f025ad2f3dfa38fd401aa64414bd561f
---
 tox.ini | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/tox.ini b/tox.ini
index 701cf167e..514962de5 100644
--- a/tox.ini
+++ b/tox.ini
@@ -122,7 +122,13 @@ commands = {posargs}
 deps = -r{toxinidir}/doc/requirements.txt
 
 [testenv:docs]
-deps = -r{toxinidir}/doc/requirements.txt
+# Do not remove requirements.txt from deps list as without it
+# upper constraints will not be used for deps listed in requirements.txt
+# and may cause issues
+deps =
+  -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}
+  -r{toxinidir}/doc/requirements.txt
+  -r{toxinidir}/requirements.txt
 allowlist_externals =
   rm
 commands =