From 8628e52de7412e57e13238ad1ba7113deb6a2e1b Mon Sep 17 00:00:00 2001 From: Vishakha Agarwal Date: Tue, 21 Jul 2020 18:24:58 +0530 Subject: Add name and enabled param in ListDomain parser when doing openstack domain list --name xyz_id, and openstack domain 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 --name and --enabled in parser of ListDomain. [1]https://docs.openstack.org/api-ref/identity/v3/?expanded=list-domains-detail#list-domains Change-Id: I3cdb511d3c7059ddfb802ca025188d8976c9302c --- openstackclient/identity/v3/domain.py | 23 ++++++++++++++++++++++- 1 file changed, 22 insertions(+), 1 deletion(-) (limited to 'openstackclient/identity') diff --git a/openstackclient/identity/v3/domain.py b/openstackclient/identity/v3/domain.py index e33fce05..e0bd1020 100644 --- a/openstackclient/identity/v3/domain.py +++ b/openstackclient/identity/v3/domain.py @@ -126,9 +126,30 @@ class DeleteDomain(command.Command): class ListDomain(command.Lister): _description = _("List domains") + def get_parser(self, prog_name): + parser = super(ListDomain, self).get_parser(prog_name) + parser.add_argument( + '--name', + metavar='', + help=_('The domain name'), + ) + parser.add_argument( + '--enabled', + dest='enabled', + action='store_true', + help=_('The domains that are enabled will be returned'), + ) + return parser + def take_action(self, parsed_args): + kwargs = {} + if parsed_args.name: + kwargs['name'] = parsed_args.name + if parsed_args.enabled: + kwargs['enabled'] = True + columns = ('ID', 'Name', 'Enabled', 'Description') - data = self.app.client_manager.identity.domains.list() + data = self.app.client_manager.identity.domains.list(**kwargs) return (columns, (utils.get_item_properties( s, columns, -- cgit v1.2.1