summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVishakha Agarwal <agarwalvishakha18@gmail.com>2018-10-19 12:29:00 +0530
committerDean Troyer <dtroyer@gmail.com>2018-11-08 11:19:24 -0600
commit0d764cdb5a1f9a8b113721634fcc8948ddd82c8d (patch)
treec2867b89c23f45210e0eee283b77e8dfca559d63
parent02f492ae4cc52ed2642f42bc0c1e5e417ebfe527 (diff)
downloadpython-openstackclient-3.17.0.tar.gz
Add project param in LimitList parser3.17.0
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
-rw-r--r--doc/source/cli/command-objects/limit.rst5
-rw-r--r--openstackclient/identity/v3/limit.py13
-rw-r--r--openstackclient/tests/unit/identity/v3/test_limit.py3
-rw-r--r--releasenotes/notes/bug-1798744-5512256baf4dc633.yaml4
4 files changed, 23 insertions, 2 deletions
diff --git a/doc/source/cli/command-objects/limit.rst b/doc/source/cli/command-objects/limit.rst
index 71cf2a42..46da6c1b 100644
--- a/doc/source/cli/command-objects/limit.rst
+++ b/doc/source/cli/command-objects/limit.rst
@@ -73,6 +73,7 @@ List project-specific limits
[--service <service>]
[--resource-name <resource-name>]
[--region <region>]
+ [--project <project>]
.. option:: --service <service>
@@ -86,6 +87,10 @@ List project-specific limits
The region name to filter the response by
+.. option:: --project <project>
+
+ List resource limits associated with project
+
limit show
----------
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 = (
diff --git a/releasenotes/notes/bug-1798744-5512256baf4dc633.yaml b/releasenotes/notes/bug-1798744-5512256baf4dc633.yaml
new file mode 100644
index 00000000..1dd01ba0
--- /dev/null
+++ b/releasenotes/notes/bug-1798744-5512256baf4dc633.yaml
@@ -0,0 +1,4 @@
+---
+features:
+ - Add ``--project`` option to ``limit list`` command.
+ [Bug `1798744 <https://bugs.launchpad.net/python-openstackclient/+bug/1798744>`_]