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 -- Mailing list: https://launchpad.net/~yahoo-eng-team Post to : [email protected] Unsubscribe : https://launchpad.net/~yahoo-eng-team More help : https://help.launchpad.net/ListHelp

