baha/skyline-apiserver
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: Fix member use has no snapshot_id filter for volumes

Browse Source 
We use system session to filter volumes with snapshot_id.
The snapshot_id filter is non-available for member user.

Change-Id: I6424f117d28378a8eb1dcafa02e6c50029f62015
This commit is contained in:
Boxiang Zhu 2022-08-23 15:06:08 +08:00
parent 7f0a0818f7
commit aa77306945
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
skyline_apiserver/api/v1/extension.py
View File

@ -840,14 +840,18 @@ async def list_volume_snapshots(
), ),
) )
for i in range(0, len(snapshot_ids), STEP): for i in range(0, len(snapshot_ids), STEP):
# Here we use system_session to filter volume with snapshot_id list.
# So we need to set all_tenants as True to filter volume from
# all volumes. Otherwise, we just filter volume from the user
# of system_session.
tasks.append( tasks.append(
cinder.list_volumes( cinder.list_volumes(
profile=profile, profile=profile,
session=current_session, session=get_system_session(),
global_request_id=x_openstack_request_id, global_request_id=x_openstack_request_id,
search_opts={ search_opts={
"snapshot_id": snapshot_ids[i : i + STEP], "snapshot_id": snapshot_ids[i : i + STEP],
"all_tenants": all_projects, "all_tenants": True,
}, },
), ),
) )