openstack/python-openstackclient
Code Issues Proposed changes

Merge "Identity: Deprecate --region None in limits/registered limits"

Browse Source
This commit is contained in:
Zuul 2025-02-12 00:22:23 +00:00 committed by Gerrit Code Review
commit 0c2dee5e1f
2 changed files with 33 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
openstackclient/identity/v3/limit.py
View File

@ -90,6 +90,13 @@ class CreateLimit(command.ShowOne):
region = common_utils.get_resource( region = common_utils.get_resource(
identity_client.regions, parsed_args.region identity_client.regions, parsed_args.region
) )
else:
self.log.warning(
_(
"Passing 'None' to indicate no region is deprecated. "
"Instead, don't pass --region."
)
)
limit = identity_client.limits.create( limit = identity_client.limits.create(
project, project,
@ -158,6 +165,14 @@ class ListLimit(command.Lister):
region = common_utils.get_resource( region = common_utils.get_resource(
identity_client.regions, parsed_args.region identity_client.regions, parsed_args.region
) )
else:
self.log.warning(
_(
"Passing 'None' to indicate no region is deprecated. "
"Instead, don't pass --region."
)
)
project = None project = None
if parsed_args.project: if parsed_args.project:
project = utils.find_resource( project = utils.find_resource(

18
openstackclient/identity/v3/registered_limit.py
View File

@ -81,6 +81,13 @@ class CreateRegisteredLimit(command.ShowOne):
region = common_utils.get_resource( region = common_utils.get_resource(
identity_client.regions, parsed_args.region identity_client.regions, parsed_args.region
) )
else:
self.log.warning(
_(
"Passing 'None' to indicate no region is deprecated. "
"Instead, don't pass --region."
)
)
registered_limit = identity_client.registered_limits.create( registered_limit = identity_client.registered_limits.create(
service, service,
@ -182,6 +189,13 @@ class ListRegisteredLimit(command.Lister):
region = common_utils.get_resource( region = common_utils.get_resource(
identity_client.regions, parsed_args.region identity_client.regions, parsed_args.region
) )
else:
self.log.warning(
_(
"Passing 'None' to indicate no region is deprecated. "
"Instead, don't pass --region."
)
)
registered_limits = identity_client.registered_limits.list( registered_limits = identity_client.registered_limits.list(
service=service, service=service,
@ -280,6 +294,10 @@ class SetRegisteredLimit(command.ShowOne):
region = common_utils.get_resource( region = common_utils.get_resource(
identity_client.regions, parsed_args.region identity_client.regions, parsed_args.region
) )
else:
self.log.warning(
_("Passing 'None' to indicate no region is deprecated.")
)
registered_limit = identity_client.registered_limits.update( registered_limit = identity_client.registered_limits.update(
parsed_args.registered_limit_id, parsed_args.registered_limit_id,