diff options
| author | Steve Martinelli <stevemar@ca.ibm.com> | 2015-10-18 16:59:01 -0400 |
|---|---|---|
| committer | Gilles Dubreuil <gilles@redhat.com> | 2015-11-12 02:56:49 +0000 |
| commit | c079e137748d4914c0da56e2b148eb4495b41db4 (patch) | |
| tree | e91829cf5091e185af8e4a2007a9365f3a1ca9bf /openstackclient/identity | |
| parent | 6611f3781be44e7981f319073b2cb737bc0d0d0c (diff) | |
| download | python-openstackclient-c079e137748d4914c0da56e2b148eb4495b41db4.tar.gz | |
better format remote IDs for identity providers
remote-ids are a list, and we should format these values as such,
rather than python representations of lists/arrays.
Closes-Bug: 1478995
Change-Id: Ia6ced0fab2435b8cb486822c676c0dee32613abe
Diffstat (limited to 'openstackclient/identity')
| -rw-r--r-- | openstackclient/identity/v3/identity_provider.py | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/openstackclient/identity/v3/identity_provider.py b/openstackclient/identity/v3/identity_provider.py index 27982a9d..50bed1f6 100644 --- a/openstackclient/identity/v3/identity_provider.py +++ b/openstackclient/identity/v3/identity_provider.py @@ -88,6 +88,8 @@ class CreateIdentityProvider(show.ShowOne): enabled=parsed_args.enabled) idp._info.pop('links', None) + remote_ids = utils.format_list(idp._info.pop('remote_ids', [])) + idp._info['remote_ids'] = remote_ids return zip(*sorted(six.iteritems(idp._info))) @@ -221,9 +223,11 @@ class ShowIdentityProvider(show.ShowOne): @utils.log_method(log) def take_action(self, parsed_args): identity_client = self.app.client_manager.identity - identity_provider = utils.find_resource( + idp = utils.find_resource( identity_client.federation.identity_providers, parsed_args.identity_provider) - identity_provider._info.pop('links', None) - return zip(*sorted(six.iteritems(identity_provider._info))) + idp._info.pop('links', None) + remote_ids = utils.format_list(idp._info.pop('remote_ids', [])) + idp._info['remote_ids'] = remote_ids + return zip(*sorted(six.iteritems(idp._info))) |