<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Sat, Dec 12, 2015 at 10:27 PM Jeremy Stanley <<a href="mailto:fungi@yuggoth.org">fungi@yuggoth.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 2015-12-12 19:00:23 +0000 (+0000), Yuriy Taraday wrote:<br>
> I think it should be a good first step in right direction. For example,<br>
> with today's issue it would break gate for tempest itself only since all<br>
> other jobs would have preinstalled tox reverted to one mentioned in<br>
> upper-constraints.<br>
[...]<br>
<br>
Other way around. It would force DevStack to downgrade tox if the<br>
existing version on the worker were higher. Pretty much no other<br>
jobs install tox during the job, so they rely entirely on the one<br>
present on the system being correct and an entry for tox in<br>
upper-constraints.txt wouldn't help them at all, whether they're<br>
using that file to constrain their requirements lists or not (since<br>
tox is not present in any of our projects' requirements lists).<br></blockquote><div><br></div><div>By "other" jobs I meant all jobs that use devstack to install tempest. That's seem to be all jobs in all projects except probably tempest itself.</div><div><br></div><div>As for jobs that don't use devstack but only run tox, I suggest us to add a step to adjust tox version according to upper-constraints as well.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Also, the constraints list is built from pip installing everything<br>
in global-requirements.txt into a virtualenv, so if tox is not a<br>
direct or transitive requirement then it will end up dropped from<br>
upper-constraints.txt on the next automated proposal in review.<br></blockquote><div><br></div><div>Ok, will fix that in my CR. </div></div></div>