From 0d764cdb5a1f9a8b113721634fcc8948ddd82c8d Mon Sep 17 00:00:00 2001
From: Vishakha Agarwal <agarwalvishakha18@gmail.com>
Date: Fri, 19 Oct 2018 12:29:00 +0530
Subject: Add project param in LimitList parser

when doing openstack limit list --project xyz_id,
CLI raising error unrecognized arguments, whereas
in api-ref document [1], user can pass project_id
as query param.This addresses the above issue, by
adding param --project in parser of LimitList.

[1] https://developer.openstack.org/api-ref/identity/v3/index.html

Change-Id: If4644cc99a3803f61f4a688b828aeb73977fc0dd
Closes-Bug: #1798744
---
 openstackclient/identity/v3/limit.py                 | 13 ++++++++++++-
 openstackclient/tests/unit/identity/v3/test_limit.py |  3 ++-
 2 files changed, 14 insertions(+), 2 deletions(-)

(limited to 'openstackclient')

diff --git a/openstackclient/identity/v3/limit.py b/openstackclient/identity/v3/limit.py
index c6f1cb1f..57d1dfd6 100644
--- a/openstackclient/identity/v3/limit.py
+++ b/openstackclient/identity/v3/limit.py
@@ -116,6 +116,11 @@ class ListLimit(command.Lister):
             metavar='<region>',
             help=_('Region for the registered limit to affect.'),
         )
+        parser.add_argument(
+            '--project',
+            metavar='<project>',
+            help=_('List resource limits associated with project'),
+        )
         return parser
 
     def take_action(self, parsed_args):
@@ -131,11 +136,17 @@ class ListLimit(command.Lister):
             region = utils.find_resource(
                 identity_client.regions, parsed_args.region
             )
+        project = None
+        if parsed_args.project:
+            project = utils.find_resource(
+                identity_client.projects, parsed_args.project
+            )
 
         limits = identity_client.limits.list(
             service=service,
             resource_name=parsed_args.resource_name,
-            region=region
+            region=region,
+            project=project
         )
 
         columns = (
diff --git a/openstackclient/tests/unit/identity/v3/test_limit.py b/openstackclient/tests/unit/identity/v3/test_limit.py
index 44c0358d..e5cd87b8 100644
--- a/openstackclient/tests/unit/identity/v3/test_limit.py
+++ b/openstackclient/tests/unit/identity/v3/test_limit.py
@@ -362,7 +362,8 @@ class TestLimitList(TestLimit):
         columns, data = self.cmd.take_action(parsed_args)
 
         self.limit_mock.list.assert_called_with(
-            service=None, resource_name=None, region=None
+            service=None, resource_name=None, region=None,
+            project=None
         )
 
         collist = (
-- 
cgit v1.2.1