commit 42e4ba7aa29869aecece62632f8afa27b651ee80
author Ghanshyam <gmaan@ghanshyammann.com> Mon May 12 13:10:30 2025 -0700
committer Ghanshyam <gmaan@ghanshyammann.com> Wed Jun 25 20:17:58 2025 +0000
tree 462765abc67f7e4260d4f5fc3c4e7c34034be2c0
parent 905cb14d8d6d7225356d88a1db7f03afcc54a435

Fix project manager user to be created in samae project

Tempest dynamic credential creates the project per test class and adds requested role users to
the same project. Project members, project admins, and project readers share the same project,
but the project manager does not share the same project; instead, the project ends up being
created as a separate project.

Fixing the same and also add the unit test.

Closes-Bug: #2110507

Change-Id: I63a0395025272202bb62851b76e8ec22f5f6db24

tempest/lib/common/dynamic_creds.py

diff --git a/tempest/lib/common/dynamic_creds.py b/tempest/lib/common/dynamic_creds.py
index 6c90938..1815dc6 100644
--- a/tempest/lib/common/dynamic_creds.py
+++ b/tempest/lib/common/dynamic_creds.py
@@ -384,8 +384,9 @@
                                                        subnet_id=subnet_id)
 
     def _get_project_id(self, credential_type, scope):
-        same_creds = [['admin'], ['member'], ['reader']]
-        same_alt_creds = [['alt_admin'], ['alt_member'], ['alt_reader']]
+        same_creds = [['admin'], ['manager'], ['member'], ['reader']]
+        same_alt_creds = [['alt_admin'], ['alt_manager'],
+                          ['alt_member'], ['alt_reader']]
         search_in = []
         if credential_type in same_creds:
             search_in = same_creds

tempest/tests/lib/common/test_dynamic_creds.py

diff --git a/tempest/tests/lib/common/test_dynamic_creds.py b/tempest/tests/lib/common/test_dynamic_creds.py
index 4c2ea30..4122db3 100644
--- a/tempest/tests/lib/common/test_dynamic_creds.py
+++ b/tempest/tests/lib/common/test_dynamic_creds.py
@@ -252,6 +252,7 @@
             reader_func = creds.get_project_alt_reader_creds
         else:
             admin_func = creds.get_project_admin_creds
+            manager_func = creds.get_project_manager_creds
             member_func = creds.get_project_member_creds
             reader_func = creds.get_project_reader_creds
         self._mock_assign_user_role()
@@ -286,6 +287,15 @@
         self._request_and_check_second_creds(
             creds, admin_func, member_creds, show_mock, sm_count=2)
 
+        # Now request for the project manager creds which should not create new
+        # project instead should use the project_id of member_creds already
+        # created project.
+        # TODO(gmaan): test test_alt_creds also once alt project
+        # manager is available.
+        if not test_alt_creds:
+            self._request_and_check_second_creds(
+                creds, manager_func, member_creds, show_mock, sm_count=3)
+
     def test_creds_within_same_project(self):
         self._creds_within_same_project()