Public bug reported: flake8 new release 3.8.0 added new checks and gate pep8 job start failing. hacking 3.0.1 fix the pinning of flake8 to avoid bringing in a new version with new checks.
Though it is fixed in latest hacking but 2.0 and 3.0 has cap for flake8 as <4.0.0 which mean flake8 new version 3.9.0 can also break the pep8 job if new check are added. To avoid similar gate break in future, we need to bump the hacking min version. - http://lists.openstack.org/pipermail/openstack- discuss/2020-May/014828.html https://review.opendev.org/#/c/727347/ changing the hacing version, but some related error in the code, see below: pep8 run-test-pre: PYTHONHASHSEED='1704528896' pep8 finish: run-test-pre after 0.00 seconds pep8 start: run-test pep8 run-test: commands[0] | bash tools/flake8wrap.sh setting PATH=/home/zuul/src/opendev.org/openstack/nova/.tox/shared/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games [4453] /home/zuul/src/opendev.org/openstack/nova$ /bin/bash tools/flake8wrap.sh Running flake8 on all files ./nova/tests/unit/virt/vmwareapi/test_driver_api.py:126:14: H202: assertRaises Exception too broad ./nova/tests/unit/compute/test_compute_mgr.py:6264:9: E306 expected 1 blank line before a nested definition, found 0 ./nova/api/openstack/compute/servers.py:104:23: E741 ambiguous variable name 'l' ./nova/db/sqlalchemy/migrate_repo/versions/330_enforce_mitaka_online_migrations.py:48:35: E711 comparison to None should be 'if cond is None:' ERROR: InvocationError for command /bin/bash tools/flake8wrap.sh (exited with code 1) pep8 finish: run-test after 70.95 seconds pep8 start: run-test-post pep8 finish: run-test-post after 0.00 seconds ___________________________________ summary ____________________________________ ERROR: pep8: commands failed ** Affects: nova Importance: Low Assignee: Brin Zhang (zhangbailin) Status: In Progress ** Changed in: nova Importance: Undecided => Low ** Changed in: nova Status: New => Confirmed ** Changed in: nova Assignee: (unassigned) => Brin Zhang (zhangbailin) -- 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/1878317 Title: pep8 error caused by hacking version < 3.0.1 Status in OpenStack Compute (nova): In Progress Bug description: flake8 new release 3.8.0 added new checks and gate pep8 job start failing. hacking 3.0.1 fix the pinning of flake8 to avoid bringing in a new version with new checks. Though it is fixed in latest hacking but 2.0 and 3.0 has cap for flake8 as <4.0.0 which mean flake8 new version 3.9.0 can also break the pep8 job if new check are added. To avoid similar gate break in future, we need to bump the hacking min version. - http://lists.openstack.org/pipermail/openstack- discuss/2020-May/014828.html https://review.opendev.org/#/c/727347/ changing the hacing version, but some related error in the code, see below: pep8 run-test-pre: PYTHONHASHSEED='1704528896' pep8 finish: run-test-pre after 0.00 seconds pep8 start: run-test pep8 run-test: commands[0] | bash tools/flake8wrap.sh setting PATH=/home/zuul/src/opendev.org/openstack/nova/.tox/shared/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games [4453] /home/zuul/src/opendev.org/openstack/nova$ /bin/bash tools/flake8wrap.sh Running flake8 on all files ./nova/tests/unit/virt/vmwareapi/test_driver_api.py:126:14: H202: assertRaises Exception too broad ./nova/tests/unit/compute/test_compute_mgr.py:6264:9: E306 expected 1 blank line before a nested definition, found 0 ./nova/api/openstack/compute/servers.py:104:23: E741 ambiguous variable name 'l' ./nova/db/sqlalchemy/migrate_repo/versions/330_enforce_mitaka_online_migrations.py:48:35: E711 comparison to None should be 'if cond is None:' ERROR: InvocationError for command /bin/bash tools/flake8wrap.sh (exited with code 1) pep8 finish: run-test after 70.95 seconds pep8 start: run-test-post pep8 finish: run-test-post after 0.00 seconds ___________________________________ summary ____________________________________ ERROR: pep8: commands failed To manage notifications about this bug go to: https://bugs.launchpad.net/nova/+bug/1878317/+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
