<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Apr 20, 2018 at 7:33 AM, Jim Rollenhagen <span dir="ltr"><<a href="mailto:jim@jimrollenhagen.com" target="_blank">jim@jimrollenhagen.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On Thu, Apr 19, 2018 at 3:21 PM, Doug Hellmann <span dir="ltr"><<a href="mailto:doug@doughellmann.com" target="_blank">doug@doughellmann.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="m_6692350212284803429h5">
<br>
</div></div>Reading through that log more carefully, I see an early attempt to pin<br>
pycodestyle <= 2.3.1 [1], followed later by pycodestyle == 2.4.0 being<br>
pulled in as a dependency of flake8-import-order==0.12 when neutron's<br>
test-requirements.txt is installed [2]. Then later when ironic's<br>
test-requirements.txt is installed pycodestyle is downgraded to 2.3.1<br>
[3].<br>
<br>
Reproducing those install & downgrade steps, I see that pycodestyle<br>
2.4.0 claims to own pep8.py but pycodestyle 2.3.1 does not [4]. So that<br>
explains why pep8 is not re-installed when pycodestyle is downgraded.<br></blockquote><div><br></div></span><div>Aha, interesting! That's a fun one. :) </div><span class=""><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I think the real problem here is that we have linter dependencies listed<br>
in the test-requirements.txt files for our projects, and they are<br>
somehow being installed without the constraints.</blockquote><div><br></div></span><div>This is because they're in the blacklist, right?</div><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I don't think they need<br>
to be installed for devstack at all, so one way to fix it would be to<br>
move those dependencies to the tox.ini section for running pep8, or to<br>
have devstack look at the blacklisted packages and skip installing them.<br></blockquote><div><br></div></span><div>Yeah, seems like either would work. With the latter, would devstack edit these out of test-requirements.txt before installing, I presume? The former seems less hacky, I'll proceed with that unless folks have objections.</div></div></div></div></blockquote><div><br></div><div>Although... this would need to be done in every project installed from source during the devstack run. I'm going to look into doing this in devstack instead to avoid spending all day moving patches.</div><div><br></div><div>// jim</div></div></div></div>