openstack/tempest
Code Issues Proposed changes

Use service role also in test_assisted_volume_snapshots

Browse Source 
Assisted volume snapshots API is service-to-service API
and nova is changing its default permission to 'service'
role
- https://review.opendev.org/c/openstack/nova/+/892635

To pass test on new defaults as well as old defaults
testing jobs, we need to use service role also in
the Tempest creds. Once nova change is merged then we
can use only service role to access this API.

Needed-By: https://review.opendev.org/c/openstack/nova/+/892635
Change-Id: I2c4368b8e2378842919fd16fa6f07505605f283a
This commit is contained in:
Ghanshyam Mann 2023-08-23 22:23:16 -07:00
parent 28336ece6f
commit 1470ddcee8
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
tempest/api/compute/admin/test_assisted_volume_snapshots.py
View File

@ -26,6 +26,13 @@ class VolumesAssistedSnapshotsTest(base.BaseV2ComputeAdminTest):
create_default_network = True
# TODO(gmann): Remove the admin access to service user
# once nova change the default of this API to service
# role. To merge the nova changing the policy default
# we need to use token with admin as well as service
# role and later we can use only service token.
credentials = ['primary', 'admin', ['service_user', 'admin', 'service']]
@classmethod
def skip_checks(cls):
super(VolumesAssistedSnapshotsTest, cls).skip_checks()
@ -37,7 +44,7 @@ class VolumesAssistedSnapshotsTest(base.BaseV2ComputeAdminTest):
def setup_clients(cls):
super(VolumesAssistedSnapshotsTest, cls).setup_clients()
cls.assisted_v_client = (
cls.os_admin.assisted_volume_snapshots_client)
cls.os_service_user.assisted_volume_snapshots_client)
cls.volumes_client = cls.os_admin.volumes_client_latest
cls.servers_client = cls.os_admin.servers_client