Merge pull request #414 from cho4036/develop

bugfix. fix delete tks role API
openinfradev · Apr 22, 2024 · a2f0337 · a2f0337
2 parents da53a5a + 4f12238
commit a2f0337
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/internal/usecase/role.go b/internal/usecase/role.go
@@ -57,7 +57,11 @@ func (r RoleUsecase) GetTksRole(ctx context.Context, organizationId string, id s
 }
 
 func (r RoleUsecase) DeleteTksRole(ctx context.Context, organizationId string, id string) error {
-	err := r.kc.DeleteGroup(ctx, organizationId, id)
+	role, err := r.repo.GetTksRole(ctx, organizationId, id)
+	if err != nil {
+		return err
+	}
+	err = r.kc.DeleteGroup(ctx, organizationId, role.Name+"@"+organizationId)
 	if err != nil {
 		return err
 	}