Skip to content

Commit 871082b

Browse files
Remove manual rollback for failed generated repositories (#20639)
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: wxiaoguang <[email protected]>
1 parent fba2055 commit 871082b

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

services/repository/template.go

Lines changed: 0 additions & 7 deletions
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
)
@@ -100,11 +98,6 @@ func GenerateRepository(doer, owner *user_model.User, templateRepo *repo_model.R
10098

10199
return nil
102100
}); err != nil {
103-
if generateRepo != nil && generateRepo.ID > 0 {
104-
if errDelete := models.DeleteRepository(doer, owner.ID, generateRepo.ID); errDelete != nil {
105-
log.Error("Rollback deleteRepository: %v", errDelete)
106-
}
107-
}
108101
return nil, err
109102
}
110103

0 commit comments

Comments
 (0)