Merge branch 'teamrole'

nais · Feb 20, 2025 · 1d55260 · 1d55260
2 parents d5d4a02 + 5699947
commit 1d55260
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/internal/reconcilers/google/gcp/reconciler_test.go b/internal/reconcilers/google/gcp/reconciler_test.go
@@ -329,7 +329,7 @@ func TestReconcile(t *testing.T) {
 					t.Errorf("expected role id %q, got %q", expectedTeamRoleId, payload.RoleId)
 				}
 
-				if expected := 32; payload.Role.IncludedPermissions != nil && len(payload.Role.IncludedPermissions) != expected {
+				if expected := 31; payload.Role.IncludedPermissions != nil && len(payload.Role.IncludedPermissions) != expected {
 					t.Errorf("expected %d permissions, got %d", expected, len(payload.Role.IncludedPermissions))
 				}
 

diff --git a/internal/reconcilers/google/gcp/role.go b/internal/reconcilers/google/gcp/role.go
@@ -50,7 +50,6 @@ func (r *googleGcpReconciler) createTeamRole(ctx context.Context, projectId stri
 			"cloudsql.users.list",
 			"cloudsql.users.update",
 			"resourcemanager.projects.get",
-			"resourcemanager.projects.list",
 			"resourcemanager.projects.getIamPolicy",
 			"storage.buckets.get",
 			"storage.buckets.getIamPolicy",