openstack/python-cinderclient
Code Issues Proposed changes

Honour all_tenants in consistency group list

Browse Source 
Although consistency groups are deprecated, the CLI command and
API support still exists. Currently it accepts the --all-tenant
parameter but doesn't pass it to the API request.
This patch corrects that behavior.

The test for consistency group were removed with the removal of
v2 API and also no tests existed for consistency group list so
no point in adding them at this point.

Change-Id: Ib3557efa50941d75d7f7f0cac01404f5c2db4526
This commit is contained in:
Rajat Dhasmana 2023-09-05 17:14:56 +05:30
parent f1f14dfbb7
commit cf2c27cc97
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
cinderclient/v3/shell_base.py
View File

@ -1975,7 +1975,9 @@ def do_unmanage(cs, args):
help='Shows details for all tenants. Admin only.')
def do_consisgroup_list(cs, args):
"""Lists all consistency groups."""
consistencygroups = cs.consistencygroups.list()
search_opts = {'all_tenants': args.all_tenants}
consistencygroups = cs.consistencygroups.list(search_opts=search_opts)
columns = ['ID', 'Status', 'Name']
shell_utils.print_list(consistencygroups, columns)