From e7418382ccbc26aa1ff7a283e5703887babcd946 Mon Sep 17 00:00:00 2001
From: Joe Gordon <joe.gordon0@gmail.com>
Date: Wed, 7 Jan 2015 12:20:33 -0800
Subject: [PATCH] Fix and re-gate on H306

H306, alphabetical imports, was mistakenly skipped in
f96ec4411ce89606cf52211061003c14306dcfa1

Change-Id: I16db4600107b1f8f54b9f46a8552331612859346
---
 nova/db/sqlalchemy/types.py | 2 +-
 nova/test.py                | 2 +-
 nova/tests/fixtures.py      | 3 ++-
 tox.ini                     | 3 +--
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/nova/db/sqlalchemy/types.py b/nova/db/sqlalchemy/types.py
index 9c9e36f41..87408c33f 100644
--- a/nova/db/sqlalchemy/types.py
+++ b/nova/db/sqlalchemy/types.py
@@ -15,8 +15,8 @@
 
 """Custom SQLAlchemy types."""
 
-from oslo.utils import netutils
 import netaddr
+from oslo.utils import netutils
 from sqlalchemy.dialects import postgresql
 from sqlalchemy import types
 
diff --git a/nova/test.py b/nova/test.py
index b1cea6116..b69c26fe4 100644
--- a/nova/test.py
+++ b/nova/test.py
@@ -26,8 +26,8 @@ eventlet.monkey_patch(os=False)
 
 import copy
 import inspect
-import mock
 import logging
+import mock
 import os
 
 import fixtures
diff --git a/nova/tests/fixtures.py b/nova/tests/fixtures.py
index 652fae89a..b21e723ed 100644
--- a/nova/tests/fixtures.py
+++ b/nova/tests/fixtures.py
@@ -26,9 +26,10 @@ import warnings
 import fixtures
 from oslo.config import cfg
 from oslo.messaging import conffixture as messaging_conffixture
+
 from nova.db import migration
-from nova import rpc
 from nova.db.sqlalchemy import api as session
+from nova import rpc
 from nova import service
 
 _TRUE_VALUES = ('True', 'true', '1', 'yes')
diff --git a/tox.ini b/tox.ini
index 17249918c..51fcbeadc 100644
--- a/tox.ini
+++ b/tox.ini
@@ -67,10 +67,9 @@ commands = python setup.py build_sphinx
 # New from hacking 0.9: E129, E131, H407, H405
 # E251 Skipped due to https://github.com/jcrocholl/pep8/issues/301
 # New from hacking 0.10: H238
-# H306 was mistakenly ignored, TODO to re-add
 # Temporarily skip W292 (new in hacking 0.10)
 
-ignore = E121,E122,E123,E124,E125,E126,E127,E128,E129,E131,E251,H405,H238,H306,W292
+ignore = E121,E122,E123,E124,E125,E126,E127,E128,E129,E131,E251,H405,H238,W292
 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'.