openstack/python-keystoneclient
Code Issues Proposed changes

Merge "Using common method 'bool_from_string' from oslo strutils"

Browse Source
This commit is contained in:
Jenkins
2014-02-13 14:41:23 +00:00
committed by Gerrit Code Review
parent ef69e08078 a7a55cb652
commit 8b0c5eea3c
2 changed files with 5 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
keystoneclient/utils.py
View File

@@ -135,13 +135,6 @@ def isunauthenticated(f):
return getattr(f, 'unauthenticated', False) return getattr(f, 'unauthenticated', False)
def string_to_bool(arg):
if isinstance(arg, bool):
return arg
return arg.strip().lower() in ('t', 'true', 'yes', '1')
def hash_signed_token(signed_text): def hash_signed_token(signed_text):
hash_ = hashlib.md5() hash_ = hashlib.md5()
hash_.update(signed_text) hash_.update(signed_text)

9
keystoneclient/v2_0/shell.py
View File

@@ -21,6 +21,7 @@ import sys
import six import six
from keystoneclient.openstack.common import strutils
from keystoneclient import utils from keystoneclient import utils
from keystoneclient.v2_0 import client from keystoneclient.v2_0 import client
@@ -86,7 +87,7 @@ def do_user_create(kc, args):
tenant_id = None tenant_id = None
user = kc.users.create(args.name, args.passwd, args.email, user = kc.users.create(args.name, args.passwd, args.email,
tenant_id=tenant_id, tenant_id=tenant_id,
enabled=utils.string_to_bool(args.enabled)) enabled=strutils.bool_from_string(args.enabled))
utils.print_dict(user._info) utils.print_dict(user._info)
@@ -105,7 +106,7 @@ def do_user_update(kc, args):
if args.email is not None: if args.email is not None:
kwargs['email'] = args.email kwargs['email'] = args.email
if args.enabled: if args.enabled:
kwargs['enabled'] = utils.string_to_bool(args.enabled) kwargs['enabled'] = strutils.bool_from_string(args.enabled)
if not len(kwargs): if not len(kwargs):
print("User not updated, no arguments present.") print("User not updated, no arguments present.")
@@ -197,7 +198,7 @@ def do_tenant_create(kc, args):
"""Create new tenant.""" """Create new tenant."""
tenant = kc.tenants.create(args.name, tenant = kc.tenants.create(args.name,
description=args.description, description=args.description,
enabled=utils.string_to_bool(args.enabled)) enabled=strutils.bool_from_string(args.enabled))
utils.print_dict(tenant._info) utils.print_dict(tenant._info)
@@ -217,7 +218,7 @@ def do_tenant_update(kc, args):
if args.description is not None: if args.description is not None:
kwargs.update({'description': args.description}) kwargs.update({'description': args.description})
if args.enabled: if args.enabled:
kwargs.update({'enabled': utils.string_to_bool(args.enabled)}) kwargs.update({'enabled': strutils.bool_from_string(args.enabled)})
if kwargs == {}: if kwargs == {}:
print("Tenant not updated, no arguments present.") print("Tenant not updated, no arguments present.")