Start using pyflakes

Instead of globally disabling pyflakes warnings,
disable only those that occur frequently and
fix the rest. Enable gating on those.

Change-Id: I774d809ebcda2339b30c104b031211a3b2c491bd

18 files changed, 18 insertions, 56 deletions

diff --git a/tox.ini b/tox.ini
index 9332b14..ba425f2 100644
--- a/tox.ini
+++ b/tox.ini
@@ -32,6 +32,6 @@ commands = python setup.py testr --coverage --testr-args='{posargs}'
     sphinx-build -b html {toxinidir}/docs/source {envtmpdir}/html
 
 [flake8]
-ignore = E12,F,H
+ignore = E12,F821,F841,H
 show-source = True
 exclude=.venv,.git,.tox,dist,doc,*lib/python*,*egg,build
diff --git a/troveclient/__init__.py b/troveclient/__init__.py
index e3506af..f4b2d27 100644
--- a/troveclient/__init__.py
+++ b/troveclient/__init__.py
@@ -14,18 +14,18 @@
 #    under the License.
 
 
-from troveclient.accounts import Accounts
-from troveclient.databases import Databases
-from troveclient.flavors import Flavors
-from troveclient.instances import Instances
-from troveclient.hosts import Hosts
-from troveclient.management import Management
-from troveclient.management import RootHistory
-from troveclient.root import Root
-from troveclient.storage import StorageInfo
-from troveclient.users import Users
-from troveclient.versions import Versions
-from troveclient.diagnostics import DiagnosticsInterrogator
-from troveclient.diagnostics import HwInfoInterrogator
-from troveclient.client import Dbaas
-from troveclient.client import TroveHTTPClient
+from troveclient.accounts import Accounts   # noqa
+from troveclient.databases import Databases  # noqa
+from troveclient.flavors import Flavors   # noqa
+from troveclient.instances import Instances  # noqa
+from troveclient.hosts import Hosts    # noqa
+from troveclient.management import Management   # noqa
+from troveclient.management import RootHistory  # noqa
+from troveclient.root import Root   # noqa
+from troveclient.storage import StorageInfo    # noqa
+from troveclient.users import Users   # noqa
+from troveclient.versions import Versions    # noqa
+from troveclient.diagnostics import DiagnosticsInterrogator    # noqa
+from troveclient.diagnostics import HwInfoInterrogator   # noqa
+from troveclient.client import Dbaas   # noqa
+from troveclient.client import TroveHTTPClient     # noqa
diff --git a/troveclient/auth.py b/troveclient/auth.py
index 909d45b..964068d 100644
--- a/troveclient/auth.py
+++ b/troveclient/auth.py
@@ -128,23 +128,6 @@ class Auth1_1(Authenticator):
                                 "key": self.password}}
         return self._authenticate(auth_url, body, root_key='auth')
 
-        try:
-            print(resp_body)
-            self.auth_token = resp_body['auth']['token']['id']
-        except KeyError:
-            raise nova_exceptions.AuthorizationFailure()
-
-        catalog = resp_body['auth']['serviceCatalog']
-        if 'cloudDatabases' not in catalog:
-            raise nova_exceptions.EndpointNotFound()
-        endpoints = catalog['cloudDatabases']
-        for endpoint in endpoints:
-            if self.region_name is None or \
-                endpoint['region'] == self.region_name:
-                self.management_url = endpoint['publicURL']
-                return
-        raise nova_exceptions.EndpointNotFound()
-
 
 class RaxAuthenticator(Authenticator):
 
diff --git a/troveclient/cli.py b/troveclient/cli.py
index 487a45c..22045cb 100644
--- a/troveclient/cli.py
+++ b/troveclient/cli.py
@@ -18,8 +18,6 @@
 Trove Command line tool
 """
 
-#TODO(tim.simpson): optparse is deprecated. Replace with argparse.
-import optparse
 import os
 import sys
 
diff --git a/troveclient/client.py b/troveclient/client.py
index 1409448..335daa4 100644
--- a/troveclient/client.py
+++ b/troveclient/client.py
@@ -130,7 +130,6 @@ class TroveHTTPClient(httplib2.Http):
         _logger.debug("RESP:%s %s\n", resp, body)
 
     def pretty_log(self, args, kwargs, resp, body):
-        from troveclient import common
         if not _logger.isEnabledFor(logging.DEBUG):
             return
 
@@ -283,7 +282,7 @@ class Dbaas(object):
 
     Create an instance with your creds::
 
-        >>> red = Dbaas(USERNAME, API_KEY, TENANT, AUTH_URL, SERVICE_NAME,
+        >>> red = Dbaas(USERNAME, API_KEY, TENANT, AUTH_URL, SERVICE_NAME, \
                         SERVICE_URL)
 
     Then call methods on its managers::
diff --git a/troveclient/databases.py b/troveclient/databases.py
index 09157d3..20f3cd5 100644
--- a/troveclient/databases.py
+++ b/troveclient/databases.py
@@ -2,7 +2,6 @@ from troveclient import base
 from troveclient.common import check_for_exceptions
 from troveclient.common import limit_url
 from troveclient.common import Paginated
-import exceptions
 import urlparse
 
 
diff --git a/troveclient/diagnostics.py b/troveclient/diagnostics.py
index 311a7c9..08c4977 100644
--- a/troveclient/diagnostics.py
+++ b/troveclient/diagnostics.py
@@ -14,7 +14,6 @@
 #    under the License.
 
 from troveclient import base
-import exceptions
 
 
 class Diagnostics(base.Resource):
diff --git a/troveclient/flavors.py b/troveclient/flavors.py
index 02fef33..f950bdc 100644
--- a/troveclient/flavors.py
+++ b/troveclient/flavors.py
@@ -16,10 +16,6 @@
 
 from troveclient import base
 
-import exceptions
-
-from troveclient.common import check_for_exceptions
-
 
 class Flavor(base.Resource):
     """
diff --git a/troveclient/mcli.py b/troveclient/mcli.py
index 87179c1..f4097fe 100644
--- a/troveclient/mcli.py
+++ b/troveclient/mcli.py
@@ -19,7 +19,6 @@ Trove Management Command line tool
 """
 
 import json
-import optparse
 import os
 import sys
 
diff --git a/troveclient/root.py b/troveclient/root.py
index e9768af..4940002 100644
--- a/troveclient/root.py
+++ b/troveclient/root.py
@@ -17,7 +17,6 @@ from troveclient import base
 
 from troveclient import users
 from troveclient.common import check_for_exceptions
-import exceptions
 
 
 class Root(base.ManagerWithFind):
diff --git a/troveclient/tests/test_auth.py b/troveclient/tests/test_auth.py
index 28cecb9..4c1023d 100644
--- a/troveclient/tests/test_auth.py
+++ b/troveclient/tests/test_auth.py
@@ -1,4 +1,3 @@
-import contextlib
 
 from testtools import TestCase
 from troveclient import auth
diff --git a/troveclient/tests/test_base.py b/troveclient/tests/test_base.py
index ac7318b..fc974f0 100644
--- a/troveclient/tests/test_base.py
+++ b/troveclient/tests/test_base.py
@@ -6,7 +6,6 @@ from mock import Mock
 
 from troveclient import base
 from troveclient import exceptions
-from troveclient import utils
 
 """
 Unit tests for base.py
diff --git a/troveclient/tests/test_client.py b/troveclient/tests/test_client.py
index 263316b..469178a 100644
--- a/troveclient/tests/test_client.py
+++ b/troveclient/tests/test_client.py
@@ -1,5 +1,3 @@
-import contextlib
-import os
 import logging
 import httplib2
 import time
@@ -9,7 +7,6 @@ from mock import Mock
 
 from troveclient import client
 from troveclient import exceptions
-from troveclient import utils
 
 """
 Unit tests for client.py
diff --git a/troveclient/tests/test_secgroups.py b/troveclient/tests/test_secgroups.py
index 779d01e..e85e18b 100644
--- a/troveclient/tests/test_secgroups.py
+++ b/troveclient/tests/test_secgroups.py
@@ -2,7 +2,6 @@ from testtools import TestCase
 from mock import Mock
 
 from troveclient import security_groups
-from troveclient import base
 
 """
 Unit tests for security_groups.py
diff --git a/troveclient/tests/test_utils.py b/troveclient/tests/test_utils.py
index 12ee9d0..217b596 100644
--- a/troveclient/tests/test_utils.py
+++ b/troveclient/tests/test_utils.py
@@ -1,7 +1,6 @@
 import os
 from testtools import TestCase
 from troveclient import utils
-from troveclient import versions
 
 
 class UtilsTest(TestCase):
@@ -34,7 +33,7 @@ class UtilsTest(TestCase):
         self.assertEqual('', utils.env('test_abcd'))
 
     def test_slugify(self):
-        import unicodedata
+        import unicodedata  # noqa
 
         self.assertEqual('not_unicode', utils.slugify('not_unicode'))
         self.assertEqual('unicode', utils.slugify(unicode('unicode')))
diff --git a/troveclient/users.py b/troveclient/users.py
index acf99ca..f20cbd0 100644
--- a/troveclient/users.py
+++ b/troveclient/users.py
@@ -19,7 +19,6 @@ from troveclient.common import check_for_exceptions
 from troveclient.common import limit_url
 from troveclient.common import Paginated
 from troveclient.common import quote_user_host
-import exceptions
 import urlparse
 
 
diff --git a/troveclient/utils.py b/troveclient/utils.py
index 3deb806..716e0f5 100644
--- a/troveclient/utils.py
+++ b/troveclient/utils.py
@@ -14,7 +14,6 @@
 
 import os
 import re
-import sys
 
 
 class HookableMixin(object):
diff --git a/troveclient/xml.py b/troveclient/xml.py
index d4a7451..85e30a1 100644
--- a/troveclient/xml.py
+++ b/troveclient/xml.py
@@ -1,5 +1,4 @@
 from lxml import etree
-import json
 from numbers import Number
 
 from troveclient import exceptions