Merge "PEP8 python-novaclient cleanup"

AUTHORS

@@ -37,3 +37,4 @@ Robie Basak <robie.basak@canonical.com>
 Sandy Walsh <sandy@darksecretsoftware.com>
 Unmesh Gurjar <unmesh.gurjar@vertex.co.in>
 William Wolf <throughnothing@gmail.com>
+Zhongyue Luo <lzyeval@gmail.com>

novaclient/base.py

@@ -74,7 +74,7 @@ def _list(self, url, response_key, obj_class=None, body=None):
         data = body[response_key]
         # NOTE(ja): keystone returns values as list as {'values': [ ... ]}
         #           unlike other services which just return the list...
-        if type(data) is dict:
+        if isinstance(data, dict):
             try:
                 data = data['values']
             except KeyError:

novaclient/client.py

@@ -192,7 +192,7 @@ def authenticate(self):
         magic_tuple = urlparse.urlsplit(self.auth_url)
         scheme, netloc, path, query, frag = magic_tuple
         port = magic_tuple.port
-        if port == None:
+        if port is None:
             port = 80
         path_parts = path.split('/')
         for part in path_parts:

novaclient/v1_1/quotas.py

@@ -51,7 +51,7 @@ def update(self, tenant_id, metadata_items=None,
                 'cores': cores}}
 
         for key in body['quota_set'].keys():
-            if body['quota_set'][key] == None:
+            if body['quota_set'][key] is None:
                 body['quota_set'].pop(key)
 
         self._update('/os-quota-sets/%s' % (tenant_id), body)

novaclient/v1_1/shell.py

@@ -503,7 +503,7 @@ def do_rebuild(cs, args):
     server = _find_server(cs, args.server)
     image = _find_image(cs, args.image)
 
-    if args.rebuild_password != False:
+    if args.rebuild_password is not False:
         _password = args.rebuild_password
     else:
         _password = None
@@ -1054,7 +1054,7 @@ def __init__(self, obj):
                 elif k == 'group':
                     k = 'source_group'
                     v = v.get('name')
-                if v == None:
+                if v is None:
                     v = ''
 
                 setattr(self, k, v)