openstack/python-barbicanclient
Code Issues Proposed changes

pep-8 fixes.

Browse Source
This commit is contained in:
Douglas Mendizabal
2013-09-04 17:06:48 -05:00
parent 5ec37db11e
commit c322aece44
3 changed files with 13 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
barbicanclient/client.py
View File

@@ -79,8 +79,8 @@ class Client(object):
raise ValueError('Barbican endpoint url must be provided, or '
'must be available from auth_plugin')
if tenant_id is None:
raise ValueError('Tenant ID must be provided, or must be available'
' from auth_plugin')
raise ValueError('Tenant ID must be provided, or must be'
' available from the auth_plugin')
if endpoint.endswith('/'):
self._barbican_url = endpoint[:-1]
else:

16
barbicanclient/keep.py
View File

@@ -80,10 +80,11 @@ class Keep:
help='a human-friendly name.')
create_parser.add_argument('--algorithm', '-a', default='aes',
help='the algorithm to be used with the '
'requested key (default: %(default)s).')
'requested key (default: '
'%(default)s).')
create_parser.add_argument('--bit-length', '-b', default=256,
help='the bit length of the requested secret'
' key (default: %(default)s).',
help='the bit length of the requested'
' secret key (default: %(default)s).',
type=int)
create_parser.add_argument('--mode', '-m', default='cbc',
help='the algorithmm mode to be used with '
@@ -151,8 +152,9 @@ class Keep:
action='store_true')
get_parser.add_argument('--payload_content_type', '-t',
default='text/plain',
help='the content type of the decrypted secret '
'(default: %(default)s; only used for secrets)')
help='the content type of the decrypted'
' secret (default: %(default)s; only used for'
' secrets)')
get_parser.set_defaults(func=self.get)
def _add_list_args(self):
@@ -192,8 +194,8 @@ class Keep:
args.expiration)
print order
else:
self.parser.exit(status=1, message='ERROR: create is only supported'
' for orders\n')
self.parser.exit(status=1, message='ERROR: create is only '
'supported for orders\n')
def delete(self, args):
if args.command == 'secret':

3
barbicanclient/orders.py
View File

@@ -120,7 +120,8 @@ class OrderManager(base.BaseEntityManager):
:param offset: Offset orders to begin list
:returns: list of Order objects
"""
LOG.debug('Listing orders - offest {0} limit {1}'.format(offset, limit))
LOG.debug('Listing orders - offest {0} limit {1}'.format(offset,
limit))
href = '{0}/{1}'.format(self.api.base_url, self.entity)
params = {'limit': limit, 'offset': offset}
resp = self.api.get(href, params)