openstack/python-openstackclient
Code Issues Proposed changes

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

Browse Source 
Change-Id: I37afac1185595216e868202c861d3de719b32073
This commit is contained in:
ArtofBugs
2025-01-28 09:10:22 -08:00
committed by Oria Weng
parent d2d7219231
commit 060d706bf4
2 changed files with 33 additions and 0 deletions
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(
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(
project,
@@ -158,6 +165,14 @@ class ListLimit(command.Lister):
region = common_utils.get_resource(
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
if parsed_args.project:
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(
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(
service,
@@ -182,6 +189,13 @@ class ListRegisteredLimit(command.Lister):
region = common_utils.get_resource(
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(
service=service,
@@ -280,6 +294,10 @@ class SetRegisteredLimit(command.ShowOne):
region = common_utils.get_resource(
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(
parsed_args.registered_limit_id,