]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Enable import group hacking rule
authorZhiteng Huang <zhithuang@ebaysf.com>
Fri, 27 Jun 2014 10:32:01 +0000 (18:32 +0800)
committerZhiteng Huang <zhithuang@ebaysf.com>
Thu, 14 Aug 2014 05:10:58 +0000 (13:10 +0800)
This change enables H305, H307 included in newly updated hacking 0.9
all violations are fixed.

H305, H307 will check if imports are grouped in stdlib, 3rd libs and
OpenStack components.

Change-Id: I612cae64982450a9be5207dbe26a2b52da49dac6

95 files changed:
bin/cinder-backup
bin/cinder-volume
bin/cinder-volume-usage-audit
cinder/api/contrib/hosts.py
cinder/api/contrib/volume_actions.py
cinder/api/contrib/volume_transfer.py
cinder/api/openstack/wsgi.py
cinder/api/urlmap.py
cinder/api/v1/snapshot_metadata.py
cinder/api/v1/volume_metadata.py
cinder/api/v1/volumes.py
cinder/api/v2/snapshot_metadata.py
cinder/api/v2/volumes.py
cinder/backup/api.py
cinder/backup/driver.py
cinder/backup/drivers/ceph.py
cinder/backup/drivers/swift.py
cinder/brick/local_dev/lvm.py
cinder/brick/remotefs/remotefs.py
cinder/db/sqlalchemy/migrate_repo/versions/004_volume_type_to_uuid.py
cinder/db/sqlalchemy/migrate_repo/versions/005_add_source_volume_column.py
cinder/db/sqlalchemy/migrate_repo/versions/021_add_default_quota_class.py
cinder/db/sqlalchemy/models.py
cinder/exception.py
cinder/test.py
cinder/tests/api/contrib/test_admin_actions.py
cinder/tests/api/contrib/test_backups.py
cinder/tests/api/contrib/test_qos_specs_manage.py
cinder/tests/api/contrib/test_services.py
cinder/tests/api/contrib/test_types_extra_specs.py
cinder/tests/api/contrib/test_volume_encryption_metadata.py
cinder/tests/api/contrib/test_volume_manage.py
cinder/tests/api/contrib/test_volume_type_encryption.py
cinder/tests/api/contrib/test_volume_unmanage.py
cinder/tests/api/middleware/test_faults.py
cinder/tests/api/openstack/test_wsgi.py
cinder/tests/api/v1/test_limits.py
cinder/tests/api/v2/test_limits.py
cinder/tests/backup/fake_swift_client.py
cinder/tests/fake_hp_3par_client.py
cinder/tests/fake_hp_lefthand_client.py
cinder/tests/integrated/api/client.py
cinder/tests/integrated/test_volumes.py
cinder/tests/scheduler/test_capacity_weigher.py
cinder/tests/scheduler/test_host_manager.py
cinder/tests/scheduler/test_rpcapi.py
cinder/tests/scheduler/test_volume_number_weigher.py
cinder/tests/test_backup.py
cinder/tests/test_backup_ceph.py
cinder/tests/test_backup_driver_base.py
cinder/tests/test_emc_vnxdirect.py
cinder/tests/test_glusterfs.py
cinder/tests/test_gpfs.py
cinder/tests/test_hds.py
cinder/tests/test_hds_iscsi.py
cinder/tests/test_hp3par.py
cinder/tests/test_hp_msa.py
cinder/tests/test_hplefthand.py
cinder/tests/test_huawei_hvs.py
cinder/tests/test_huawei_t_dorado.py
cinder/tests/test_ibmnas.py
cinder/tests/test_image_utils.py
cinder/tests/test_netapp.py
cinder/tests/test_netapp_eseries_iscsi.py
cinder/tests/test_netapp_nfs.py
cinder/tests/test_netapp_ssc.py
cinder/tests/test_nimble.py
cinder/tests/test_pure.py
cinder/tests/test_rbd.py
cinder/tests/test_storwize_svc.py
cinder/tests/test_volume.py
cinder/tests/test_volume_utils.py
cinder/tests/test_wsgi.py
cinder/tests/volume/drivers/netapp/test_iscsi.py
cinder/tests/windows/test_windows.py
cinder/tests/zonemanager/test_brcd_fc_san_lookup_service.py
cinder/tests/zonemanager/test_brcd_fc_zone_client_cli.py
cinder/tests/zonemanager/test_brcd_fc_zone_driver.py
cinder/tests/zonemanager/test_fc_zone_manager.py
cinder/utils.py
cinder/volume/drivers/emc/emc_smis_common.py
cinder/volume/drivers/hds/hds.py
cinder/volume/drivers/hds/hnas_backend.py
cinder/volume/drivers/hds/iscsi.py
cinder/volume/drivers/hds/nfs.py
cinder/volume/drivers/huawei/rest_common.py
cinder/volume/drivers/huawei/ssh_common.py
cinder/volume/drivers/ibm/storwize_svc/helpers.py
cinder/volume/drivers/netapp/api.py
cinder/volume/drivers/netapp/eseries/client.py
cinder/volume/drivers/prophetstor/dpl_fc.py
cinder/volume/drivers/rbd.py
cinder/volume/drivers/san/hp/hp_lefthand_rest_proxy.py
cinder/volume/drivers/vmware/read_write_util.py
tox.ini

index 1b474d3ba584540c2dd288a9d320e33883ba9c31..1c502fa47f076be192fea73ada35f05df802bd64 100755 (executable)
@@ -25,11 +25,10 @@ import warnings
 warnings.simplefilter('once', DeprecationWarning)
 
 import eventlet
+from oslo.config import cfg
 
 eventlet.monkey_patch()
 
-from oslo.config import cfg
-
 # If ../cinder/__init__.py exists, add ../ to Python search path, so that
 # it will override what happens to be installed in /usr/(local/)lib/python...
 possible_topdir = os.path.normpath(os.path.join(os.path.abspath(sys.argv[0]),
index 8a51231cf19b0776642ee17eb645f7576df63dc6..e2396c7ecf57a777fa8503688c8e343c66cdd687 100755 (executable)
 
 """Starter script for Cinder Volume."""
 
-import eventlet
 import os
 
+import eventlet
+
 if os.name == 'nt':
     # eventlet monkey patching the os module causes subprocess.Popen to fail
     # on Windows when using pipes due to missing non-blocking IO support.
index caa1dee9f1fafad2d6c078f6551d5b25aab5c9a9..09c8c428f03f909354f4de9d2cef1c0c58b98e93 100755 (executable)
@@ -54,7 +54,6 @@ if os.path.exists(os.path.join(POSSIBLE_TOPDIR, 'cinder', '__init__.py')):
 
 from cinder import i18n
 i18n.enable_lazy()
-
 from cinder import context
 from cinder import db
 from cinder.i18n import _
index 0d6dc4ea112d5c334ecd3098eb3054d486d194ce..024d8f3bca1905c832e802b37910ffb1331ca315 100644 (file)
 
 """The hosts admin extension."""
 
+from xml.parsers import expat
 
 from oslo.config import cfg
 import webob.exc
-from xml.parsers import expat
 
 from cinder.api import extensions
 from cinder.api.openstack import wsgi
index 88048d4262654de6615ac3bf0eb636d975018821..24dcb9cf47d3081b9466f550f97248045c9279b6 100644 (file)
@@ -12,9 +12,9 @@
 #   License for the specific language governing permissions and limitations
 #   under the License.
 
-import webob
 
 from oslo import messaging
+import webob
 
 from cinder.api import extensions
 from cinder.api.openstack import wsgi
index f510aa69418910cd3be83db04a7f14228e891434..05daea215b3d371ff41d626b7a426c0a1c0b51ae 100644 (file)
@@ -21,7 +21,6 @@ from cinder.api import extensions
 from cinder.api.openstack import wsgi
 from cinder.api.views import transfers as transfer_view
 from cinder.api import xmlutil
-
 from cinder import exception
 from cinder.i18n import _
 from cinder.openstack.common import log as logging
index 657291dae7961172d40b0dbf1f4556b55b82a197..9e0c2102fdd5a73fba15dd183ef121378ddd9ec0 100644 (file)
 import inspect
 import math
 import time
+from xml.dom import minidom
+from xml.parsers import expat
 
 from lxml import etree
 import six
 import webob
-from xml.dom import minidom
-from xml.parsers import expat
 
 from cinder import exception
 from cinder import i18n
index e3a9ee5fb3baf6bd9169e1d26cf998d1bdaf8328..d1f2147e48a4713d49244533bd451b16df665b33 100644 (file)
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
-import paste.urlmap
 import re
 import urllib2
 
+import paste.urlmap
+
 from cinder.api.openstack import wsgi
 from cinder.openstack.common import log as logging
 
index d1913f493fbdfaa581ee7a340bea53bc00b2451b..18c2dd4a0c580fb620b7f46fafd0fd068eb783c6 100644 (file)
 #    under the License.
 
 import webob
+from webob import exc
 
 from cinder.api import common
 from cinder.api.openstack import wsgi
 from cinder import exception
 from cinder.i18n import _
 from cinder import volume
-from webob import exc
 
 
 class Controller(wsgi.Controller):
index 8130f627ccce1fe8970cc5e7bca277551d2aa151..b4f0bb3d5d4e965df888cc141f8e79e45470c3e4 100644 (file)
 #    under the License.
 
 import webob
+from webob import exc
 
 from cinder.api import common
 from cinder.api.openstack import wsgi
 from cinder import exception
 from cinder.i18n import _
 from cinder import volume
-from webob import exc
 
 
 class Controller(wsgi.Controller):
index ebf195a89a5d3e7d4e2cea498838abcef53e6b3d..5e6968f7b319d1b3017a4a5883956f1331ab83c5 100644 (file)
@@ -16,6 +16,7 @@
 """The volumes api."""
 
 import ast
+
 import webob
 from webob import exc
 
index d1913f493fbdfaa581ee7a340bea53bc00b2451b..18c2dd4a0c580fb620b7f46fafd0fd068eb783c6 100644 (file)
 #    under the License.
 
 import webob
+from webob import exc
 
 from cinder.api import common
 from cinder.api.openstack import wsgi
 from cinder import exception
 from cinder.i18n import _
 from cinder import volume
-from webob import exc
 
 
 class Controller(wsgi.Controller):
index e218e1b5b6ce56cbab724b14062e253da5b78919..9c6f17da9ced141cb6efb8990c7b6ee19e4fe820 100644 (file)
@@ -17,6 +17,7 @@
 
 
 import ast
+
 import webob
 from webob import exc
 
index 090db2769bce79355dcf0f0fa015c4cc65de2cc1..d3c195afe07b52c0c1131ca921ed27d5320ee915 100644 (file)
@@ -19,7 +19,6 @@ Handles all requests relating to the volume backups service.
 
 
 from eventlet import greenthread
-
 from oslo.config import cfg
 
 from cinder.backup import rpcapi as backup_rpcapi
@@ -28,9 +27,8 @@ from cinder.db import base
 from cinder import exception
 from cinder.i18n import _
 from cinder.openstack.common import log as logging
-from cinder import utils
-
 import cinder.policy
+from cinder import utils
 import cinder.volume
 
 CONF = cfg.CONF
index 04c84fb28e443c6e6aeb6c68bd692832e1db51cd..c831d1df7e83141bdf94ce8516600b3436580da8 100644 (file)
 
 """Base class for all backup drivers."""
 
+from oslo.config import cfg
+
 from cinder.db import base
 from cinder import exception
 from cinder.i18n import _
 from cinder.openstack.common import jsonutils
 from cinder.openstack.common import log as logging
-from oslo.config import cfg
 
 service_opts = [
     cfg.IntOpt('backup_metadata_version', default=1,
index 2a57978b2bf661a337505622a55e384ec29df604..68c7e94e7d7d6fa17eb022154817b6b6a01e1d82 100644 (file)
@@ -42,13 +42,13 @@ was deemed the safest action to take. It is therefore recommended to always
 restore to a new volume (default).
 """
 
-import eventlet
 import fcntl
 import os
 import re
 import subprocess
 import time
 
+import eventlet
 from oslo.config import cfg
 
 from cinder.backup.driver import BackupDriver
index eedc723614ff7de008634af91159c44d4045a97c..21c35b96a6c84fdd9345799c1f469eec7fe096d9 100644 (file)
 import hashlib
 import json
 import os
-import six
 import socket
 
 import eventlet
 from oslo.config import cfg
+import six
+from swiftclient import client as swift
 
 from cinder.backup.driver import BackupDriver
 from cinder import exception
@@ -46,7 +47,6 @@ from cinder.openstack.common import excutils
 from cinder.openstack.common import log as logging
 from cinder.openstack.common import timeutils
 from cinder.openstack.common import units
-from swiftclient import client as swift
 
 
 LOG = logging.getLogger(__name__)
index 798075f7c31564efacaf8837d0e4bee1139107e8..4e2be1a104e2b8adb1bc427bde9429d9d4388276 100644 (file)
 LVM class for performing LVM operations.
 """
 
+import itertools
 import math
 import re
 
-import itertools
-
 from cinder.brick import exception
 from cinder.brick import executor
 from cinder.i18n import _
index 4b67d1e45d061ffbd3e16aa3a6ec733b36eb7c44..81bf90fe583c7abb485e353051d48cf5eba93cf6 100755 (executable)
@@ -18,6 +18,7 @@
 import hashlib
 import os
 import re
+
 import six
 
 from cinder.brick import exception
index 3cd0afb9cae5261ca3ad192b8be2abedd2b99263..c5d1ba0c34919d708bbd918104867645fb902840 100644 (file)
 
 import uuid
 
-from cinder.openstack.common import log as logging
 from migrate import ForeignKeyConstraint
 from sqlalchemy import Integer, MetaData, String, Table
 
+from cinder.openstack.common import log as logging
+
 LOG = logging.getLogger(__name__)
 
 
index 352d640a92ca8beef6e83ee82747d77c99f30361..9291bb2ee68d645e726d5c4268a36de41c15e582 100644 (file)
 #    under the License.
 
 
-from cinder.openstack.common import log as logging
 from sqlalchemy import Column
 from sqlalchemy import MetaData, String, Table
 
+from cinder.openstack.common import log as logging
+
 LOG = logging.getLogger(__name__)
 
 
index f09bf395177f5c3a3485b04e56c734e785578d48..075eaf9c2fa498b3e9462947d4182f52170b38e5 100644 (file)
 
 import datetime
 
-from cinder.i18n import _
-from cinder.openstack.common import log as logging
 from oslo.config import cfg
 from sqlalchemy import MetaData, Table
 
+from cinder.i18n import _
+from cinder.openstack.common import log as logging
+
 # Get default values via config.  The defaults will either
 # come from the default values set in the quota option
 # configuration or via cinder.conf if the user has configured
index 89ec464e61dacb3b6155460c55ecab7bf8510ab5..e2860cc1d0a11555b11e17502e632ddcb45ca7ca 100644 (file)
 SQLAlchemy models for cinder data.
 """
 
-
+from oslo.config import cfg
+from oslo.db.sqlalchemy import models
 from sqlalchemy import Column, Integer, String, Text, schema
 from sqlalchemy.ext.declarative import declarative_base
 from sqlalchemy import ForeignKey, DateTime, Boolean
 from sqlalchemy.orm import relationship, backref
-from oslo.config import cfg
-from oslo.db.sqlalchemy import models
 
 from cinder.openstack.common import timeutils
 
index c383d52d60bbab57dfedd81475757a61d9762444..0a123a7c338109876a15a59e235b568c97b9946f 100644 (file)
@@ -22,10 +22,10 @@ SHOULD include dedicated exception logging.
 
 """
 
-import six
 import sys
 
 from oslo.config import cfg
+import six
 import webob.exc
 
 from cinder.i18n import _
index 17819af3e10bc22bafc91fa4f0cfc03d56700c55..c38d47d4d31ddac053ac38c916cb7ad65844e4dd 100644 (file)
@@ -21,13 +21,13 @@ inline callbacks.
 
 """
 
+import logging
 import os
 import shutil
 import tempfile
 import uuid
 
 import fixtures
-import logging
 import mox
 from oslo.config import cfg
 from oslo.messaging import conffixture as messaging_conffixture
index e14cfd58ac978a3c45c1609ebbef566df12600d0..df8ae7c25eec4ed359d228e64ba24d8a8f53938a 100644 (file)
@@ -12,9 +12,9 @@
 
 import ast
 import tempfile
-import webob
 
 from oslo.config import cfg
+import webob
 
 from cinder.api.contrib import admin_actions
 from cinder.brick.local_dev import lvm as brick_lvm
index 0cfd9ea2c894d1033ee6d6a2032783f582fe216a..4304813bf76087146db8eb0861a92ff016097318 100644 (file)
@@ -18,9 +18,9 @@ Tests for Backup code.
 """
 
 import json
-import mock
 from xml.dom import minidom
 
+import mock
 import webob
 
 # needed for stubs to work
index 84ce76fa6a8df1fdb18e9dcb5ebff57735599883..377d4224b082c1109d82ef878404b88f3b8e8853 100644 (file)
@@ -14,9 +14,9 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
-from lxml import etree
 from xml.dom import minidom
 
+from lxml import etree
 import webob
 
 from cinder.api.contrib import qos_specs_manage
index 92806d4e3b9e40b5c5c30466336f3881794e7d08..8741f22f015c179020d9ea6305865abc2f95f182 100644 (file)
@@ -14,6 +14,8 @@
 #    under the License.
 
 
+from datetime import datetime
+
 import webob.exc
 
 from cinder.api.contrib import services
@@ -25,7 +27,6 @@ from cinder.openstack.common import timeutils
 from cinder import policy
 from cinder import test
 from cinder.tests.api import fakes
-from datetime import datetime
 
 
 fake_services_list = [{'binary': 'cinder-scheduler',
index 18a60dc2a846ad49f11d1a91b100a5c5ff4586ac..f1a7cae7e3e4f7d8fd7b503b54c2e41590352767 100644 (file)
@@ -16,9 +16,8 @@
 #    under the License.
 
 from lxml import etree
-import webob
-
 import mock
+import webob
 
 from cinder.api.contrib import types_extra_specs
 from cinder import exception
index ff8a6c58c64ef1cbb1510e59d6b68e819376b3d3..29c5dec27a71be2280295ea9df60b8a0600d0ac6 100644 (file)
@@ -14,6 +14,7 @@
 #    under the License.
 
 import json
+
 import webob
 
 from cinder.api.contrib import volume_encryption_metadata
index 3ccf99acba5a0aeb4c05b113c329b935e487ef1a..82bad3c109e5daa55101633cbf441f9831c5e778 100755 (executable)
@@ -13,7 +13,6 @@
 #   under the License.
 
 import mock
-
 import webob
 
 from cinder import context
index 3ab8a326d88d34ae4fdb4f10d0b9492074aca67b..9bf1f9852acadaa5d51889db0e2f208c25d82b43 100644 (file)
@@ -14,6 +14,7 @@
 #    under the License.
 
 import json
+
 import webob
 
 from cinder import context
index cde2791758f4142d53cfbbf2142405100b8dba53..6538cbb153a44b09eef1bd51010898ca3634602c 100755 (executable)
@@ -13,7 +13,6 @@
 #   under the License.
 
 import mock
-
 import webob
 
 from cinder import context
index 5f9eb75e269d611a38a4f1096d3ed4052461ae6f..60297cf17fea5abaaaca60f41c2ff64efc638ed2 100644 (file)
@@ -13,9 +13,9 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+import gettext
 from xml.dom import minidom
 
-import gettext
 import mock
 from oslo.i18n import _lazy
 import webob.dec
index af8658c3122ec2ce9b912ea81e58366a76ac72b4..0a39d8d96a223b08d47d3f725639e8fac9b8089e 100644 (file)
@@ -11,6 +11,7 @@
 # under the License.
 
 import inspect
+
 import webob
 
 from cinder.api.openstack import wsgi
index ee36fe22eb83818cb59a9c9bbc323aff9fe04a28..711d57320179047bd2b7761bab6ec6e108054033 100644 (file)
@@ -18,10 +18,10 @@ Tests dealing with HTTP rate-limiting.
 """
 
 import httplib
-import six
 from xml.dom import minidom
 
 from lxml import etree
+import six
 import webob
 
 from cinder.api.v1 import limits
index 6c3baa70fefa9897d2b3590187dfcbce6e486e26..5d377728030aef8b851d3e44cf0e2b599671e39f 100644 (file)
@@ -18,11 +18,11 @@ Tests dealing with HTTP rate-limiting.
 """
 
 import httplib
+from xml.dom import minidom
 
 from lxml import etree
 import six
 import webob
-from xml.dom import minidom
 
 from cinder.api.v2 import limits
 from cinder.api import views
index dc35ea7f2c5678e12aed76f2b0ae2baa8865ac7b..70ac143618a769246ade65ee5e33821974072477 100644 (file)
@@ -19,9 +19,10 @@ import os
 import socket
 import zlib
 
-from cinder.openstack.common import log as logging
 from swiftclient import client as swift
 
+from cinder.openstack.common import log as logging
+
 LOG = logging.getLogger(__name__)
 
 
index 073eb2f0018399e0ecf777ca5fb70a946166cebf..85905448c3c4bccb21e1159847453932bfeb13dd 100644 (file)
 #
 """Fake HP client for testing 3PAR without installing the client."""
 
-import mock
 import sys
 
+import mock
+
 from cinder.tests import fake_hp_client_exceptions as hpexceptions
 
 hp3par = mock.Mock()
index 9af0607cd6714c564e6c2a2832f07c18658525fc..604cc888b3837d72a67253d0a9ac677d3a541282 100644 (file)
 #
 """Fake HP client for testing LeftHand without installing the client."""
 
-import mock
 import sys
 
+import mock
+
 from cinder.tests import fake_hp_client_exceptions as hpexceptions
 
 hplefthand = mock.Mock()
index eeb599ef52a8c593ed1c169e3bf7f7c56f028a72..b8d5970180fcbf3997fefc26b3d74876afc9e699 100644 (file)
@@ -14,7 +14,6 @@
 
 import netaddr
 import requests
-
 import six.moves.urllib.parse as urlparse
 
 from cinder.i18n import _
index 9bdc872dbf9b6c0c64408f8f8997410923632e39..4ab011882d5902aa478bd6262030f154b845df1b 100644 (file)
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
-import testtools
 import time
 
+import testtools
+
 from cinder.openstack.common import log as logging
 from cinder import service
 from cinder.tests import fake_driver
index 28e266ef8f004323931b4b1d78b1635fe96d2fe7..b1fe53bc5cfafd06237eccd675fa2a47534f9533 100644 (file)
@@ -17,7 +17,6 @@ Tests For Capacity Weigher.
 """
 
 import mock
-
 from oslo.config import cfg
 
 from cinder import context
index 7140fa1a7bfe69afe43d34f71fbb6bcf2f71f118..c8c7f5747cf9a3d1370ae974c1677867905938b4 100644 (file)
@@ -17,7 +17,6 @@ Tests For HostManager
 """
 
 import mock
-
 from oslo.config import cfg
 
 from cinder import exception
index 2622a2296f5ffca7f8750a518944b992152ead83..b48c64751bbe20bf413d191336a17c7fe033f484 100644 (file)
@@ -18,8 +18,8 @@ Unit Tests for cinder.scheduler.rpcapi
 """
 
 import copy
-import mock
 
+import mock
 from oslo.config import cfg
 
 from cinder import context
index ca596cf048422c88d4b7d53bd02169d53a86e218..14832b31e3238fcdaa382bc668702c36e29dc311 100644 (file)
@@ -17,7 +17,6 @@ Tests For Volume Number Weigher.
 """
 
 import mock
-
 from oslo.config import cfg
 
 from cinder import context
index dd0b604a6596f14cf074becff98484ac6dba1d99..ade2463cc58af116e2238bdb02578d4470e4c4d4 100644 (file)
@@ -17,9 +17,9 @@ Tests for Backup code.
 
 """
 
-import mock
 import tempfile
 
+import mock
 from oslo.config import cfg
 
 from cinder import context
index fb905ebee138bdaf082403c90ee7ca3c93c20334..75fc6cc1179392fc03e6192e659693f6e106476a 100644 (file)
 
 import contextlib
 import hashlib
-import mock
 import os
-import six
 import tempfile
 import uuid
 
+import mock
+import six
+
 from cinder.backup import driver
 from cinder.backup.drivers import ceph
 from cinder import context
index 7a05590a6984d7a3438a7b3b95e5aeb4c35e73cd..3ad2348891fda67a68396839a53f41736140d7f4 100644 (file)
 #    under the License.
 """ Tests for the backup service base driver. """
 
-import mock
 import uuid
 
+import mock
+
 from cinder.backup import driver
 from cinder import context
 from cinder import db
index 217e0c69f89a06ff0ebdb719aded7f03b21ed17c..5910ff441940c34bbc0f7759149dc93d1bf118aa 100644 (file)
 #    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
 #    License for the specific language governing permissions and limitations
 #    under the License.
-import mock
+
 import os
 import re
 
+import mock
+
 from cinder import exception
 from cinder.openstack.common import processutils
 from cinder import test
index 608493882f192d36fe45d6113983f66319d25d3f..093cb4f62c1dd2d9b20c34a83862d581abe7eac7 100644 (file)
 
 import contextlib
 import errno
-import mock
 import os
 import tempfile
 import time
 import traceback
 
+import mock
 import mox as mox_lib
 from mox import IgnoreArg
 from mox import IsA
index bc71600f2e68bd910273aac3b138b5055a1d1870..f3a3c71fc35cba4901255bba74417e110d2b5f3c 100644 (file)
 # License for the specific language governing permissions and limitations
 # under the License.
 
-import mock
 import os
 import tempfile
 
+import mock
 from oslo.config import cfg
 
 from cinder import context
index e2cd2d64004e5dddc95741b8ad0be4abcf4c5087..65a2bf7889d104255f61f48211a678b3b0058b73 100644 (file)
 Self test for Hitachi Unified Storage (HUS) platform.
 """
 
-import mox
 import os
 import tempfile
 
+import mox
+
 from cinder import test
 from cinder.volume import configuration as conf
 from cinder.volume.drivers.hds import hds
index 48bcaffac58a43ac8d62ad59fadcb909a81b5a87..3b8e1cf010d1141d124f2ac07f71eb681651b8c9 100644 (file)
@@ -23,11 +23,10 @@ import tempfile
 
 import mock
 
+from cinder.openstack.common import log as logging
 from cinder import test
 from cinder.volume import configuration as conf
 from cinder.volume.drivers.hds import iscsi
-
-from cinder.openstack.common import log as logging
 LOG = logging.getLogger(__name__)
 
 HNASCONF = """<?xml version="1.0" encoding="UTF-8" ?>
index 2533b60c4e12052a441ecc46220fd0da8cf81b2b..86f7374f5813b51c801bfe997d5587a478195708 100644 (file)
@@ -22,7 +22,6 @@ from cinder import exception
 from cinder.openstack.common import log as logging
 from cinder.openstack.common import units
 from cinder import test
-
 from cinder.tests import fake_hp_3par_client as hp3parclient
 from cinder.volume.drivers.san.hp import hp_3par_fc as hpfcdriver
 from cinder.volume.drivers.san.hp import hp_3par_iscsi as hpdriver
index 0418b6f32be5f4ddf3d0c8adb4d90283577f8378..205689bdb1d9f4e14519036fb38800e8bf5ba369 100644 (file)
 
 """Unit tests for OpenStack Cinder HP MSA driver."""
 
+import urllib2
+
 import lxml.etree as etree
 import mock
-import urllib2
 
 from cinder import exception
 from cinder import test
index 5528730e16fd0a2d823a7bcce53f543cc783c157..e0ec9e89d2665c9526c6a294a817c5d41a769978 100644 (file)
@@ -21,7 +21,6 @@ from cinder import exception
 from cinder.openstack.common import log as logging
 from cinder.openstack.common import units
 from cinder import test
-
 from cinder.tests import fake_hp_lefthand_client as hplefthandclient
 from cinder.volume.drivers.san.hp import hp_lefthand_iscsi
 from cinder.volume.drivers.san.hp import hp_lefthand_rest_proxy
index 5af0354447dca4f2f27a0c54dabe0a11e3a67eaf..7ea5f76a99a4e0c76208bd0726b447ac8adb74c3 100644 (file)
@@ -19,14 +19,14 @@ Unit Tests for Huawei HVS volume drivers.
 """
 
 import json
-import mox
 import os
 import shutil
 import tempfile
 import time
-
 from xml.dom.minidom import Document
 
+import mox
+
 from cinder import exception
 from cinder import test
 from cinder.volume import configuration as conf
index 6f25f006e642df1841eb0ec96428b70b3634703b..36d84648bc5a3e592ff6d984c9c7dc4eae3b84bb 100644 (file)
 Unit Tests for Huawei T and Dorado volume drivers.
 """
 
-import mox
 import os
 import shutil
 import socket
 import tempfile
 import time
-
 from xml.dom.minidom import Document
 from xml.etree import ElementTree as ET
 
+import mox
+
 from cinder import context
 from cinder import exception
 from cinder import ssh_utils
index bff55d3d06f2de32e201ee0858f172b37a33f9f1..d6b4892fa1fa37743edec1287547098edcb98e56 100644 (file)
@@ -21,7 +21,6 @@ Tests for the IBM NAS family (SONAS, Storwize V7000 Unified).
 """
 
 import mock
-
 from oslo.config import cfg
 
 from cinder import context
index 048b88f560184c8389fb02463e61967d0329b928..c26f1707e8183a624926cf52c9539c5ae2d0295e 100644 (file)
 """Unit tests for image utils."""
 
 import contextlib
-import mock
-import mox
 import tempfile
 
+import mock
+import mox
 from oslo.config import cfg
 
 from cinder import context
index f8c7d4eb7cbbde93c8161f99bb3e72a2f2657d39..74ff95047c2662f46d8f66fc907f3845250eeb36 100644 (file)
@@ -20,8 +20,8 @@ Tests for NetApp volume driver
 
 import BaseHTTPServer
 import httplib
-from lxml import etree
 
+from lxml import etree
 import six
 
 from cinder import exception
index 00d793f2b80104aa19c88a4ccb6c080fcc0db7b2..ab03c6d78ac5a64b0d1594ade61ec0f5d5318805 100644 (file)
@@ -17,8 +17,9 @@ Tests for NetApp e-series iscsi volume driver.
 """
 
 import json
-import mock
 import re
+
+import mock
 import requests
 
 from cinder import exception
index 15a4067e649f1027bd2b145cd5946bf3c3b41cb3..9915a2331e87e80ce543795ecbc26d383cee7d8f 100644 (file)
 #    under the License.
 """Unit tests for the NetApp-specific NFS driver module."""
 
+import os
+
 from lxml import etree
 import mock
 import mox
 from mox import IgnoreArg
 from mox import IsA
-import os
 
 from cinder import context
 from cinder import exception
index 8fd26c3d05507b5457bf918223cc583217e6efd5..11ea22892073b57eb07575cfcbad60923c7c7398 100644 (file)
@@ -18,9 +18,9 @@
 import BaseHTTPServer
 import copy
 import httplib
+
 from lxml import etree
 from mox import IgnoreArg
-
 import six
 
 from cinder import exception
index bef14aa77331cbffc5b291d35b794870029970df..9096c9ac37392f63d91aac5adf82eb0d54bc7231 100644 (file)
@@ -14,7 +14,6 @@
 #    under the License.
 
 import mock
-
 from oslo.config import cfg
 
 from cinder import exception
index cf91837202c132600755ff3296b2297c8b4582d3..612e49d20f2b5e4828706671d7c0a921f8378645 100644 (file)
 #    under the License.
 
 import json
-import mock
 import urllib2
 
+import mock
+
 from cinder import exception
 from cinder.openstack.common import processutils
 from cinder.openstack.common import units
index 10f447c92d47a562647032cf3cd48fc221bc573c..8ba6a519f8957dc562ace005d570c12de3975228 100644 (file)
 
 
 import math
-import mock
 import os
 import tempfile
 
+import mock
+
 from cinder import db
 from cinder import exception
 from cinder.i18n import _
index 4b26d515af12aaaed36612a375d8771575a302a3..972739c9d1b6bef7dc4bbf35935c6d256b1beacc 100644 (file)
 Tests for the IBM Storwize family and SVC volume driver.
 """
 
-import mock
 import random
 import re
 import time
 
+import mock
+
 from cinder import context
 from cinder import exception
 from cinder.i18n import _
index 4cc95b0d5d7d0874f6cb09a4aa3b560ac4ad7eeb..538f1e8c1375c1dc874ecc0bf574fda08d660ae7 100644 (file)
@@ -20,7 +20,6 @@ Tests for Volume Code.
 
 import contextlib
 import datetime
-import mock
 import os
 import shutil
 import socket
@@ -28,6 +27,7 @@ from sys import platform
 import tempfile
 
 import eventlet
+import mock
 import mox
 from oslo.config import cfg
 from stevedore import extension
index 5a6bffc708204c04987c50df38f318001b81a9ad..61ab079516b104faa3cf92fbb44bc62da15f3e81 100644 (file)
 
 """Tests For miscellaneous util methods used with volume."""
 
-import mock
 import os
 import re
 
+import mock
 from oslo.config import cfg
 
 from cinder import context
index 214e5c5566c0d6a1ea69c6a6bc05ccbb3e1abea8..77bc0827b8a590a524e4bd9243dec0794da3f4c9 100644 (file)
 
 """Unit tests for `cinder.wsgi`."""
 
-import mock
 import os.path
 import tempfile
 import urllib2
 
+import mock
 from oslo.config import cfg
 from oslo.i18n import _lazy
 import testtools
index dd9e83f2a844845e5ed9de1511261267d81eab0f..04ded7fc15e2a9fa0af113f14542440f9ea2fc9a 100644 (file)
 Mock unit tests for the NetApp iSCSI driver
 """
 
-import mock
 import uuid
 
+import mock
+
 from cinder import test
 import cinder.volume.drivers.netapp.api as ntapi
 import cinder.volume.drivers.netapp.iscsi as ntap_iscsi
index c92e77cbb2c2da4322bcb88263bea0d812c96949..11e45bd6474cc14f8e40f595b0526fd86609bd74 100644 (file)
@@ -22,15 +22,12 @@ import os
 import shutil
 import tempfile
 
-from oslo.config import cfg
-
 import mox
-
-from cinder import test
+from oslo.config import cfg
 
 from cinder.image import image_utils
-
 from cinder.openstack.common import fileutils
+from cinder import test
 from cinder.tests.windows import db_fakes
 from cinder.volume import configuration as conf
 from cinder.volume.drivers.windows import constants
index 524c3f3eb6c65761f6a9a4afd2ce0ffab085139d..e138d452a026ca0e85530136da8b7616a057d9c2 100644 (file)
@@ -20,9 +20,8 @@
 """Unit tests for brcd fc san lookup service."""
 
 import mock
-import paramiko
-
 from oslo.config import cfg
+import paramiko
 
 from cinder import exception
 from cinder.openstack.common import log as logging
index 4f0bc7c83a804ceea79967a23ff55b166023b627..09f20f2e84efaa4f343520a3eecf60e4f057eefd 100644 (file)
@@ -20,6 +20,7 @@
 """Unit tests for brcd fc zone client cli."""
 
 import mock
+from mock import patch
 
 from cinder import exception
 from cinder.openstack.common import log as logging
@@ -28,7 +29,6 @@ from cinder import test
 from cinder.zonemanager.drivers.brocade.brcd_fc_zone_client_cli \
     import BrcdFCZoneClientCLI
 import cinder.zonemanager.drivers.brocade.fc_zone_constants as ZoneConstant
-from mock import patch
 
 LOG = logging.getLogger(__name__)
 
index 810ca76cef2b2d0314efef1bcfed7eab4b3ff1f5..f3cec3a4d44f6fb5193471eb22800ef22cd278e5 100644 (file)
@@ -20,9 +20,8 @@
 """Unit tests for Brocade fc zone driver."""
 
 import mock
-import paramiko
-
 from oslo.config import cfg
+import paramiko
 
 from cinder import exception
 from cinder.i18n import _
index 7d04ab23de818e06786d486dcb85027cdda97104..11cf92e7f7c6a115ea7f8488fd383626e5f05bb6 100644 (file)
 """Unit tests for FC Zone Manager."""
 
 import mock
+from mock import Mock
 
 from cinder import exception
 from cinder import test
 from cinder.volume import configuration as conf
 from cinder.zonemanager.drivers.fc_zone_driver import FCZoneDriver
 from cinder.zonemanager import fc_zone_manager
-from mock import Mock
 
 fabric_name = 'BRCD_FAB_3'
 init_target_map = {'10008c7cff523b01': ['20240002ac000a50']}
index 9fe1924d94b73a6c9890ad85ed01c05df21e0c3a..c0b24532b8238e51e8b92c3088fba6f8f54fd7f6 100644 (file)
@@ -29,15 +29,15 @@ import shutil
 import stat
 import sys
 import tempfile
-
-from oslo.config import cfg
-import six
 from xml.dom import minidom
 from xml.parsers import expat
 from xml import sax
 from xml.sax import expatreader
 from xml.sax import saxutils
 
+from oslo.config import cfg
+import six
+
 from cinder.brick.initiator import connector
 from cinder import exception
 from cinder.i18n import _
index 77262d720eb29c5f93b43c97ed4ab48c5d6c7cea..dc16cf6c79a13ebad38960b63f49928d96a1793d 100644 (file)
@@ -21,9 +21,9 @@ It supports VNX and VMAX arrays.
 """
 
 import time
+from xml.dom.minidom import parseString
 
 from oslo.config import cfg
-from xml.dom.minidom import parseString
 
 from cinder import exception
 from cinder.i18n import _
index 57f7b618b84a2e486eca31e7a382d18790cbd8b4..f08498f54dd57e3d7ec2237beb6edc67a7a810d5 100644 (file)
 iSCSI Cinder Volume driver for Hitachi Unified Storage (HUS) platform.
 """
 
-from oslo.config import cfg
 from xml.etree import ElementTree as ETree
 
+from oslo.config import cfg
+
 from cinder import exception
 from cinder.i18n import _
 from cinder.openstack.common import excutils
index 58075d72a66ba357cf7bcf5098bb111094dce54a..10b9a103452ec6716e3126a34f9cea97359804db 100644 (file)
 Hitachi Unified Storage (HUS-HNAS) platform. Backend operations.
 """
 
+import re
+
 from cinder.openstack.common import log as logging
 from cinder.openstack.common import units
 from cinder import utils
-import re
 
 LOG = logging.getLogger("cinder.volume.driver")
 
index 2a604d24501fc84447b544da08b285dce2806abc..f4350a56d2dd2acb0ba20a903ee6be42f84a26d1 100644 (file)
 iSCSI Cinder Volume driver for Hitachi Unified Storage (HUS-HNAS) platform.
 """
 
-from oslo.config import cfg
 from xml.etree import ElementTree as ETree
 
+from oslo.config import cfg
+
 from cinder import exception
 from cinder.i18n import _
 from cinder.openstack.common import excutils
index 2cca52355ce81d89c89cccf225259a21bb86713a..c68a6f976575d079533b0b85b615e3bace4926d4 100644 (file)
@@ -19,9 +19,9 @@ Volume driver for HDS HNAS NFS storage.
 
 import os
 import time
+from xml.etree import ElementTree as ETree
 
 from oslo.config import cfg
-from xml.etree import ElementTree as ETree
 
 from cinder import exception
 from cinder.i18n import _
index 262a9321cc149314842d5d4e0a336c15deea0bd2..fca545d504d2eb40fcc1514c11603ad10dfc3229 100644 (file)
@@ -21,7 +21,6 @@ import json
 import time
 import urllib2
 import uuid
-
 from xml.etree import ElementTree as ET
 
 from cinder import context
index a40ac3f84176e36a5c98c190ce6bd29b50c5acbb..8706811c5dd71f798b2d8df88273aa67530bf05c 100644 (file)
@@ -24,7 +24,6 @@ import re
 import socket
 import threading
 import time
-
 from xml.etree import ElementTree as ET
 
 from cinder import context
index 127dfcfb7d077e299bd6641bb88b9ad040ad6da6..99694c988cfbd1ad4dbc1c5ecf523ef16193c524 100644 (file)
 
 import random
 import re
-import six
 import unicodedata
 
 from eventlet import greenthread
+import six
 
 from cinder import context
 from cinder import exception
index 35ac582e597f9f8674e04b6fed7ff32dd1c63d14..9f5f744722f9ef7ec954c7c97238a0f789e50882 100644 (file)
@@ -19,9 +19,10 @@ NetApp api for ONTAP and OnCommand DFM.
 Contains classes required to issue api calls to ONTAP and OnCommand DFM.
 """
 
-from lxml import etree
 import urllib2
 
+from lxml import etree
+
 from cinder.i18n import _
 from cinder.openstack.common import log as logging
 
index 09ac9986c2ecbb99b2ef227b997f7a4ccab1c91a..52b376a2f5a8d8b3928b9b2d3cc6c46b29ea5615 100644 (file)
@@ -17,8 +17,8 @@ Client classes for web services.
 """
 
 import json
-import requests
 
+import requests
 import six.moves.urllib.parse as urlparse
 
 from cinder import exception
index a03ef126aadd5a63441943f97eee2a489a019746..8b040be3f9ce585e4823f2e32048b39bb67e5865 100644 (file)
@@ -14,6 +14,7 @@
 #    under the License.
 
 import errno
+
 import six
 
 from cinder import exception
index 9b67b8768e449fe19e32b4016978bcde4bc402e7..a55c1f666683e5ed545916e7933a8002f4828790 100644 (file)
@@ -18,11 +18,11 @@ import io
 import json
 import math
 import os
-import six
 import tempfile
 import urllib
 
 from oslo.config import cfg
+import six
 
 from cinder import exception
 from cinder.i18n import _
index 409c54cb128bf6273db3c9cf770e273aa223bd0d..fb39aec99b0eb4d918e0dce5ad905a5e31de29fd 100644 (file)
@@ -15,6 +15,8 @@
 #
 """HP LeftHand SAN ISCSI REST Proxy."""
 
+from oslo.config import cfg
+
 from cinder import context
 from cinder import exception
 from cinder.i18n import _
@@ -23,7 +25,6 @@ from cinder.openstack.common import units
 from cinder.volume.driver import ISCSIDriver
 from cinder.volume import utils
 from cinder.volume import volume_types
-from oslo.config import cfg
 
 LOG = logging.getLogger(__name__)
 
index db969aa4b5d6a696907e0f31b7971564cd860320..f7dc5c8702f631a539b4127434077b689ba394f5 100644 (file)
@@ -20,10 +20,10 @@ Collection of classes to handle image upload/download to/from Image service
 """
 
 import httplib
-import netaddr
 import urllib
 import urllib2
 
+import netaddr
 import six.moves.urllib.parse as urlparse
 
 from cinder.i18n import _
diff --git a/tox.ini b/tox.ini
index a9674a4f705e132efbfb69b234daeee9bf327081..6bc2e9ac81647f8d9a53f67e0f6c7547fce130bd 100644 (file)
--- a/tox.ini
+++ b/tox.ini
@@ -72,7 +72,7 @@ commands = {posargs}
 # F841,H302,H305,H307,H405
 
 
-ignore = E251,E265,H302,H305,H307,H402,H405,H803,H904
+ignore = E251,E265,H302,H402,H405,H803,H904
 builtins = _
 exclude = .git,.venv,.tox,dist,tools,doc,common,*egg,build