diff options
| author | Jamie Lennox <jamielennox@redhat.com> | 2014-04-04 08:09:43 +1000 |
|---|---|---|
| committer | Jamie Lennox <jamielennox@redhat.com> | 2014-04-04 08:09:43 +1000 |
| commit | 022b6d95d167405fa6534680c8a7fe449b35ce77 (patch) | |
| tree | a5946810a71acc78f9df8e66cd1a1c91997f8e3a /openstackclient/identity | |
| parent | ee22070473dab8bfa3e89d47f1f5a77918c2b026 (diff) | |
| download | python-openstackclient-022b6d95d167405fa6534680c8a7fe449b35ce77.tar.gz | |
Pass arguments to v3 keystoneclient by kwarg
Keystoneclient has added the positional decorator which emits a warning
if arguments aren't passed by keyword. This means we are getting
warnings in certain places in openstackclient.
Change-Id: Ic5446cd6f122cbb56fce543011386d53bc31fe18
Closes-Bug: #1302199
Diffstat (limited to 'openstackclient/identity')
| -rw-r--r-- | openstackclient/identity/v3/credential.py | 6 | ||||
| -rw-r--r-- | openstackclient/identity/v3/domain.py | 4 | ||||
| -rw-r--r-- | openstackclient/identity/v3/endpoint.py | 20 | ||||
| -rw-r--r-- | openstackclient/identity/v3/group.py | 2 | ||||
| -rw-r--r-- | openstackclient/identity/v3/policy.py | 2 | ||||
| -rw-r--r-- | openstackclient/identity/v3/project.py | 4 | ||||
| -rw-r--r-- | openstackclient/identity/v3/role.py | 2 | ||||
| -rw-r--r-- | openstackclient/identity/v3/service.py | 6 | ||||
| -rw-r--r-- | openstackclient/identity/v3/user.py | 2 |
9 files changed, 24 insertions, 24 deletions
diff --git a/openstackclient/identity/v3/credential.py b/openstackclient/identity/v3/credential.py index b82825f0..93f67f6c 100644 --- a/openstackclient/identity/v3/credential.py +++ b/openstackclient/identity/v3/credential.py @@ -68,9 +68,9 @@ class CreateCredential(show.ShowOne): else: project = None credential = identity_client.credentials.create( - user_id, - parsed_args.type, - parsed_args.data, + user=user_id, + type=parsed_args.type, + blob=parsed_args.data, project=project) return zip(*sorted(six.iteritems(credential._info))) diff --git a/openstackclient/identity/v3/domain.py b/openstackclient/identity/v3/domain.py index 1e9a4a2a..a74b12e2 100644 --- a/openstackclient/identity/v3/domain.py +++ b/openstackclient/identity/v3/domain.py @@ -61,8 +61,8 @@ class CreateDomain(show.ShowOne): self.log.debug('take_action(%s)' % parsed_args) identity_client = self.app.client_manager.identity domain = identity_client.domains.create( - parsed_args.name, - parsed_args.description, + name=parsed_args.name, + description=parsed_args.description, enabled=parsed_args.enabled, ) diff --git a/openstackclient/identity/v3/endpoint.py b/openstackclient/identity/v3/endpoint.py index 055f9fe2..64334c63 100644 --- a/openstackclient/identity/v3/endpoint.py +++ b/openstackclient/identity/v3/endpoint.py @@ -73,11 +73,11 @@ class CreateEndpoint(show.ShowOne): parsed_args.service) endpoint = identity_client.endpoints.create( - service.id, - parsed_args.url, - parsed_args.interface, - parsed_args.region, - parsed_args.enabled + service=service.id, + url=parsed_args.url, + interface=parsed_args.interface, + region=parsed_args.region, + enabled=parsed_args.enabled ) info = {} @@ -192,11 +192,11 @@ class SetEndpoint(command.Command): identity_client.endpoints.update( endpoint.id, - service.id, - parsed_args.url, - parsed_args.interface, - parsed_args.region, - parsed_args.enabled + service=service.id, + url=parsed_args.url, + interface=parsed_args.interface, + region=parsed_args.region, + enabled=parsed_args.enabled ) return diff --git a/openstackclient/identity/v3/group.py b/openstackclient/identity/v3/group.py index 6c059b5d..38d810cb 100644 --- a/openstackclient/identity/v3/group.py +++ b/openstackclient/identity/v3/group.py @@ -133,7 +133,7 @@ class CreateGroup(show.ShowOne): else: domain = None group = identity_client.groups.create( - parsed_args.name, + name=parsed_args.name, domain=domain, description=parsed_args.description) diff --git a/openstackclient/identity/v3/policy.py b/openstackclient/identity/v3/policy.py index a760d8cd..249ba1ee 100644 --- a/openstackclient/identity/v3/policy.py +++ b/openstackclient/identity/v3/policy.py @@ -52,7 +52,7 @@ class CreatePolicy(show.ShowOne): identity_client = self.app.client_manager.identity policy = identity_client.policies.create( - blob, type=parsed_args.type + blob=blob, type=parsed_args.type ) return zip(*sorted(six.iteritems(policy._info))) diff --git a/openstackclient/identity/v3/project.py b/openstackclient/identity/v3/project.py index d3618fb3..36787bb0 100644 --- a/openstackclient/identity/v3/project.py +++ b/openstackclient/identity/v3/project.py @@ -88,8 +88,8 @@ class CreateProject(show.ShowOne): kwargs = parsed_args.property.copy() project = identity_client.projects.create( - parsed_args.name, - domain, + name=parsed_args.name, + domain=domain, description=parsed_args.description, enabled=enabled, **kwargs diff --git a/openstackclient/identity/v3/role.py b/openstackclient/identity/v3/role.py index 05bdbbfc..664a05dc 100644 --- a/openstackclient/identity/v3/role.py +++ b/openstackclient/identity/v3/role.py @@ -155,7 +155,7 @@ class CreateRole(show.ShowOne): self.log.debug('take_action(%s)' % parsed_args) identity_client = self.app.client_manager.identity - role = identity_client.roles.create(parsed_args.name) + role = identity_client.roles.create(name=parsed_args.name) return zip(*sorted(six.iteritems(role._info))) diff --git a/openstackclient/identity/v3/service.py b/openstackclient/identity/v3/service.py index 7e3bfc6b..63997d1c 100644 --- a/openstackclient/identity/v3/service.py +++ b/openstackclient/identity/v3/service.py @@ -64,9 +64,9 @@ class CreateService(show.ShowOne): enabled = False service = identity_client.services.create( - parsed_args.name, - parsed_args.type, - enabled, + name=parsed_args.name, + type=parsed_args.type, + enabled=enabled, ) return zip(*sorted(six.iteritems(service._info))) diff --git a/openstackclient/identity/v3/user.py b/openstackclient/identity/v3/user.py index 060eeca7..a5209020 100644 --- a/openstackclient/identity/v3/user.py +++ b/openstackclient/identity/v3/user.py @@ -107,7 +107,7 @@ class CreateUser(show.ShowOne): parsed_args.password = utils.get_password(self.app.stdin) user = identity_client.users.create( - parsed_args.name, + name=parsed_args.name, domain=domain_id, default_project=project_id, password=parsed_args.password, |
