Skip to content

Commit f7c874c

Browse files
Remove manual rollback for failed generated repositories (#20639) (#20762)
Generating repositories from a template is done inside a transaction. Manual rollback on error is not needed and it always results in error "repository does not exist". Co-authored-by: Gabriel Vasile <[email protected]>
1 parent d19c2c9 commit f7c874c

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

services/repository/template.go

-7
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,10 @@ package repository
77
import (
88
"context"
99

10-
"code.gitea.io/gitea/models"
1110
"code.gitea.io/gitea/models/db"
1211
issues_model "code.gitea.io/gitea/models/issues"
1312
repo_model "code.gitea.io/gitea/models/repo"
1413
user_model "code.gitea.io/gitea/models/user"
15-
"code.gitea.io/gitea/modules/log"
1614
"code.gitea.io/gitea/modules/notification"
1715
repo_module "code.gitea.io/gitea/modules/repository"
1816
)
@@ -95,11 +93,6 @@ func GenerateRepository(doer, owner *user_model.User, templateRepo *repo_model.R
9593

9694
return nil
9795
}); err != nil {
98-
if generateRepo != nil && generateRepo.ID > 0 {
99-
if errDelete := models.DeleteRepository(doer, owner.ID, generateRepo.ID); errDelete != nil {
100-
log.Error("Rollback deleteRepository: %v", errDelete)
101-
}
102-
}
10396
return nil, err
10497
}
10598

0 commit comments

Comments
 (0)