Merge "Add project param in LimitList parser"
This commit is contained in:
commit
097b45686e
@ -73,6 +73,7 @@ List project-specific limits
|
|||||||
[--service <service>]
|
[--service <service>]
|
||||||
[--resource-name <resource-name>]
|
[--resource-name <resource-name>]
|
||||||
[--region <region>]
|
[--region <region>]
|
||||||
|
[--project <project>]
|
||||||
|
|
||||||
.. option:: --service <service>
|
.. option:: --service <service>
|
||||||
|
|
||||||
@ -86,6 +87,10 @@ List project-specific limits
|
|||||||
|
|
||||||
The region name to filter the response by
|
The region name to filter the response by
|
||||||
|
|
||||||
|
.. option:: --project <project>
|
||||||
|
|
||||||
|
List resource limits associated with project
|
||||||
|
|
||||||
limit show
|
limit show
|
||||||
----------
|
----------
|
||||||
|
|
||||||
|
@ -116,6 +116,11 @@ class ListLimit(command.Lister):
|
|||||||
metavar='<region>',
|
metavar='<region>',
|
||||||
help=_('Region for the registered limit to affect.'),
|
help=_('Region for the registered limit to affect.'),
|
||||||
)
|
)
|
||||||
|
parser.add_argument(
|
||||||
|
'--project',
|
||||||
|
metavar='<project>',
|
||||||
|
help=_('List resource limits associated with project'),
|
||||||
|
)
|
||||||
return parser
|
return parser
|
||||||
|
|
||||||
def take_action(self, parsed_args):
|
def take_action(self, parsed_args):
|
||||||
@ -131,11 +136,17 @@ class ListLimit(command.Lister):
|
|||||||
region = utils.find_resource(
|
region = utils.find_resource(
|
||||||
identity_client.regions, parsed_args.region
|
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(
|
limits = identity_client.limits.list(
|
||||||
service=service,
|
service=service,
|
||||||
resource_name=parsed_args.resource_name,
|
resource_name=parsed_args.resource_name,
|
||||||
region=region
|
region=region,
|
||||||
|
project=project
|
||||||
)
|
)
|
||||||
|
|
||||||
columns = (
|
columns = (
|
||||||
|
@ -362,7 +362,8 @@ class TestLimitList(TestLimit):
|
|||||||
columns, data = self.cmd.take_action(parsed_args)
|
columns, data = self.cmd.take_action(parsed_args)
|
||||||
|
|
||||||
self.limit_mock.list.assert_called_with(
|
self.limit_mock.list.assert_called_with(
|
||||||
service=None, resource_name=None, region=None
|
service=None, resource_name=None, region=None,
|
||||||
|
project=None
|
||||||
)
|
)
|
||||||
|
|
||||||
collist = (
|
collist = (
|
||||||
|
4
releasenotes/notes/bug-1798744-5512256baf4dc633.yaml
Normal file
4
releasenotes/notes/bug-1798744-5512256baf4dc633.yaml
Normal file
@ -0,0 +1,4 @@
|
|||||||
|
---
|
||||||
|
features:
|
||||||
|
- Add ``--project`` option to ``limit list`` command.
|
||||||
|
[Bug `1798744 <https://bugs.launchpad.net/python-openstackclient/+bug/1798744>`_]
|
Loading…
Reference in New Issue
Block a user