From eef23489ee89ba775ced22ab39a737d88fd8f46c Mon Sep 17 00:00:00 2001 From: jichenjc Date: Sat, 19 Sep 2015 22:58:14 +0800 Subject: [PATCH] Pep8 didn't check api/openstack/common.py flake8 rule in tox.ini has following exclude rule is exclude = .venv,.git,.tox,dist,doc,*openstack/common* which excludes api/openstack/common.py Change-Id: I39b82cf90048970e7bfc6be77de2e1543130f81b Closes-Bug: 1498634 --- nova/api/openstack/common.py | 2 +- tox.ini | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/nova/api/openstack/common.py b/nova/api/openstack/common.py index 19e0f14ef631..b248ec358f37 100644 --- a/nova/api/openstack/common.py +++ b/nova/api/openstack/common.py @@ -101,7 +101,7 @@ _STATE_MAP = { 'default': 'VERIFY_RESIZE', # Note(maoy): the OS API spec 1.1 doesn't have CONFIRMING_RESIZE # state so we comment that out for future reference only. - #task_states.RESIZE_CONFIRMING: 'CONFIRMING_RESIZE', + # task_states.RESIZE_CONFIRMING: 'CONFIRMING_RESIZE', task_states.RESIZE_REVERTING: 'REVERT_RESIZE', }, vm_states.PAUSED: { diff --git a/tox.ini b/tox.ini index 35691660466d..cebf1d238fb8 100644 --- a/tox.ini +++ b/tox.ini @@ -174,7 +174,7 @@ commands = bandit -c bandit.yaml -r nova -n 5 -ll # E251 Skipped due to https://github.com/jcrocholl/pep8/issues/301 ignore = E121,E122,E123,E124,E125,E126,E127,E128,E129,E131,E251,H405 -exclude = .venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build,tools/xenserver* +exclude = .venv,.git,.tox,dist,doc,*openstack/common/*,*lib/python*,*egg,build,tools/xenserver* # To get a list of functions that are more complex than 25, set max-complexity # to 25 and run 'tox -epep8'. # 34 is currently the most complex thing we have