Re-enable flake8-import-order

This was not carried across as it should have been in the migration to
pre-commit. Thankfully not much has drifted since.

Change-Id: I6e4d2b8109210dbaf673242b258b8018f0249b0a
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>

.pre-commit-config.yaml

@@ -62,7 +62,9 @@ repos:
     rev: 8.0.0
     hooks:
       - id: hacking
-        additional_dependencies: ['neutron-lib>=3.24.0']
+        additional_dependencies:
+          - neutron-lib>=3.24.0
+          - flake8-import-order~=0.19.2
         exclude: '^(doc|releasenotes|tools)/.*$'
   - repo: https://github.com/sphinx-contrib/sphinx-lint
     rev: v1.0.2

neutron/__init__.py

@@ -13,9 +13,10 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+import warnings
+
 # NOTE(ralonsoh): remove once the default backend is ``BackendType.THREADING``
 from oslo_service import backend as oslo_service_backend
-import warnings
 try:
     oslo_service_backend.init_backend(
         oslo_service_backend.BackendType.THREADING)
@@ -26,7 +27,7 @@ except oslo_service_backend.exceptions.BackendAlreadySelected:
 
 
 # pylint: disable=wrong-import-position
-import builtins  # noqa: E402
+import builtins  # noqa: E402,I100
 import gettext  # noqa: E402
 
 from neutron._i18n import _ as n_under  # noqa: E402

neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py

@@ -18,12 +18,12 @@ import copy
 import datetime
 import functools
 import multiprocessing
-import netaddr
 import operator
 import threading
 import types
 import uuid
 
+import netaddr
 from neutron_lib.api.definitions import portbindings
 from neutron_lib.api.definitions import provider_net
 from neutron_lib.api.definitions import segment as segment_def

tox.ini

@@ -233,7 +233,7 @@ commands = sphinx-build -W -b linkcheck doc/source doc/build/linkcheck
 # I202 Additional newline in a group of imports
 # N530 direct neutron imports not allowed
 # W504 line break after binary operator
-ignore = E126,E128,E231,E275,I202,H405,N530,W504
+ignore = E126,E128,E231,E275,H405,I202,N530,W504
 # H106: Don't put vim configuration in source files
 # H203: Use assertIs(Not)None to check for None
 # H204: Use assert(Not)Equal to check for equality