From 1e053babf4d674ac31d51dfba048704f32b558b3 Mon Sep 17 00:00:00 2001
From: Vishakha Agarwal <agarwalvishakha18@gmail.com>
Date: Fri, 24 Jul 2020 19:22:39 +0530
Subject: Add id and enabled param in ListIdentityProvider parser

when doing openstack identity provider list --name xyz_id,
and openstack identity provider list --enabled CLI raising
error unrecognized arguments, whereas in api-ref document [1],
user can pass name and enabled as optional query param. This
addresses the above issue, by adding param --id and --enabled in
parser of ListIdentityProvider.

[1] https://docs.openstack.org/api-ref/identity/v3-ext/?expanded=list-identity-providers-detail#list-identity-providers

Change-Id: I59ce3a5f54700ba5a735f0b3b4b3b73b3a8658fa
---
 openstackclient/identity/v3/identity_provider.py | 24 +++++++++++++++++++++++-
 1 file changed, 23 insertions(+), 1 deletion(-)

(limited to 'openstackclient/identity')

diff --git a/openstackclient/identity/v3/identity_provider.py b/openstackclient/identity/v3/identity_provider.py
index 2b2d9d11..7307cea0 100644
--- a/openstackclient/identity/v3/identity_provider.py
+++ b/openstackclient/identity/v3/identity_provider.py
@@ -143,10 +143,32 @@ class DeleteIdentityProvider(command.Command):
 class ListIdentityProvider(command.Lister):
     _description = _("List identity providers")
 
+    def get_parser(self, prog_name):
+        parser = super(ListIdentityProvider, self).get_parser(prog_name)
+        parser.add_argument(
+            '--id',
+            metavar='<id>',
+            help=_('The Identity Providers’ ID attribute'),
+        )
+        parser.add_argument(
+            '--enabled',
+            dest='enabled',
+            action='store_true',
+            help=_('The Identity Providers that are enabled will be returned'),
+        )
+        return parser
+
     def take_action(self, parsed_args):
         columns = ('ID', 'Enabled', 'Domain ID', 'Description')
         identity_client = self.app.client_manager.identity
-        data = identity_client.federation.identity_providers.list()
+
+        kwargs = {}
+        if parsed_args.id:
+            kwargs['id'] = parsed_args.id
+        if parsed_args.enabled:
+            kwargs['enabled'] = True
+
+        data = identity_client.federation.identity_providers.list(**kwargs)
         return (columns,
                 (utils.get_item_properties(
                     s, columns,
-- 
cgit v1.2.1