openstack/python-novaclient
Code Issues Proposed changes

Merge "PEP8 python-novaclient cleanup"

Browse Source
This commit is contained in:
Jenkins 2012-01-10 12:51:13 +00:00 committed by Gerrit Code Review
commit c747f153ec
5 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
AUTHORS
View File

@ -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>

2
novaclient/base.py
View File

@ -74,7 +74,7 @@ class Manager(utils.HookableMixin):
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:

2
novaclient/client.py
View File

@ -192,7 +192,7 @@ class HTTPClient(httplib2.Http):
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:

2
novaclient/v1_1/quotas.py
View File

@ -51,7 +51,7 @@ class QuotaSetManager(base.ManagerWithFind):
'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)

4
novaclient/v1_1/shell.py
View File

@ -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 _print_secgroup_rules(rules):
elif k == 'group':
k = 'source_group'
v = v.get('name')
if v == None:
if v is None:
v = ''
setattr(self, k, v)