[all] pep8 job failing due to flake8 3.8.0

Sean Mooney smooney at redhat.com
Tue May 12 17:27:46 UTC 2020


On Tue, 2020-05-12 at 12:23 -0500, Ghanshyam Mann wrote:
> 
> 
>  ---- On Tue, 12 May 2020 12:00:11 -0500 Brian Haley <haleyb.dev at gmail.com> wrote ----
>  > On 5/12/20 12:25 PM, Sean McGinnis wrote:
>  > > On 5/12/20 11:14 AM, Ghanshyam Mann wrote:
>  > >> Hello Everyone.
>  > >>
>  > >> You might have noticed that few or most of the projects pep8 job 
>  > >> started failing.
>  > >>
>  > >> That is because flake8 new version 3.8.0 added the new pycodestyle 
>  > >> with new rules. Hacking capped
>  > >> it with 4.0.0 version not with the minor version for 3.*.
>  > >>
>  > >> The new hacking version 3.0.1 is released which cap the flake8<3.8.0. 
>  > >> Thanks to dtantsur and stephenfin for
>  > >> tacking care of it.
>  > >>
>  > >>   To fix your pep8 job you can,
>  > >>
>  > >> - Either fix the pep8 error in code if easy and fast to fix.
>  > >>
>  > >> - Or bump the hacking minimum version to 3.0.1. I have proposed it for 
>  > >> a few projects failing pep8.
>  > >>    - 
>  > >> https://review.opendev.org/#/q/topic:hacking-fix+(status:open+OR+status:merged) 
>  > >>
>  > >>
>  > >> If pep8 job is passing then you do not need to do anything. The 
>  > >> existing hacking version cap will work fine.
>  > >>
>  > >> -gmann
>  > > 
>  > > This is also failing on some stable branches that had not moved to
>  > > hacking 3.0 yet. In this case, it may be better to add a flake8 cap to
>  > > the repo's test-requirements.txt file rather than backporting a major
>  > > bump in hacking and dealing with the need to make a lot of code changes.
>  > > 
>  > > Here is an example of that approach:
>  > > 
>  > > https://review.opendev.org/#/c/727265/
>  > 
>  > I found in the neutron stable/ussuri repo that capping flake8<3.8.0 
>  > didn't work, but capping pycodestyle did.  So that's another option.
>  > 
>  > -pycodestyle>=2.0.0 # MIT
>  > +pycodestyle>=2.0.0,<2.6.0 # MIT
>  > 
>  > https://review.opendev.org/#/c/727274/
> 
> I will say remove rhe pycodestyle from neutron test-reqruiement and let hacking
> via flake8 cap handle the compatible pycodestyle. Otherwise we end up maintaining and 
> fixing it project side for future.
just in case it was nto seen from my minimal testing if i create a tox env
with master. then i downgrade both flake8 and pycodestyle in the virtual enve the nova
tests still fail.

do i need to explcitly down grade hacking too?

.tox/venv/bin/pip install -U pycodestyle\<2.6 flake8\<3.8 and re running the test is failing for me even if i do 
find -name *.pyc -delete
to make sure there are no .pyc files cached anywhere
> 
> 
> -gmann
> 
> 
>  > 
>  > -Brian
>  > 
>  > 
> 




More information about the openstack-discuss mailing list