yahoo-eng-team team mailing list archive

[Matt Riedemann <mriedem.os@xxxxxxxxx>]

Public bug reported:

With this change in Train:

https://review.opendev.org/#/c/651553/

We're using pycodestyle rather than pep8 and pycodestyle doesn't check
for F841:

https://pycodestyle.readthedocs.io/en/latest/intro.html#error-codes

Which means things like this slip through and then fail on backports to
<train:

https://review.opendev.org/#/c/691282/

It would be good to see if we can get F841 checked on master and train
again so we don't have that difference with stable branches. I'm not
sure how trivial that is though, like if we can separately run flake8
with F841 during our pep8 runs.

This is just to track the issue but I realize it might be closed as
Won't Fix.

** Affects: nova
     Importance: Undecided
         Status: Triaged


** Tags: pep8 testing

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1849850

Title:
  F841 is not checked since Train

Status in OpenStack Compute (nova):
  Triaged

Bug description:
  With this change in Train:

  https://review.opendev.org/#/c/651553/

  We're using pycodestyle rather than pep8 and pycodestyle doesn't check
  for F841:

  https://pycodestyle.readthedocs.io/en/latest/intro.html#error-codes

  Which means things like this slip through and then fail on backports
  to <train:

  https://review.opendev.org/#/c/691282/

  It would be good to see if we can get F841 checked on master and train
  again so we don't have that difference with stable branches. I'm not
  sure how trivial that is though, like if we can separately run flake8
  with F841 during our pep8 runs.

  This is just to track the issue but I realize it might be closed as
  Won't Fix.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1849850/+subscriptions