-
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
github mirrors cause hudge slow-down #33200
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
as can be seen above, the ref stored in db is |
Fix go-gitea#33200 Co-authored-by: wxiaoguang <[email protected]>
Backport #33224 by lunny Fix #33200 Co-authored-by: Lunny Xiao <[email protected]> Co-authored-by: wxiaoguang <[email protected]>
Fix go-gitea#33200 Co-authored-by: wxiaoguang <[email protected]> (cherry picked from commit be4e961)
@lunny @wxiaoguang it looks like current main branch still has this problem, as can be seen on https://demo.gitea.com/ericLemanissier/conan-center-index/graph and on the dashboard Should I open a new issue or can you reopen this one ? |
I will take a look at this. |
follows-up be4e961 Fix go-gitea#33200 --------- Co-authored-by: Giteabot <[email protected]>
Backport #33597 by @ericLemanissier follows-up be4e961 This is the same modification as be4e961 but for force-pushes. It is needed, because `git fetch` reveals force pushes for github mirrors: ``` $ git fetch --tags origin remote: Enumerating objects: 22, done. remote: Counting objects: 100% (22/22), done. remote: Compressing objects: 100% (4/4), done. remote: Total 9 (delta 5), reused 8 (delta 5), pack-reused 0 (from 0) Unpacking objects: 100% (9/9), 1.70 KiB | 12.00 KiB/s, done. From https://github.com/conan-io/conan-center-index 729f0f1b8f..48184eddeb refs/pull/26595/head -> refs/pull/26595/head + 0c31ab60a3...1283cca9e7 refs/pull/26595/merge -> refs/pull/26595/merge (forced update) ``` Fix #33200 PS: I did not test the modification, but it is the exact same change as the last hunk in https://github.com/go-gitea/gitea/pull/33224/files#diff-bb5cdb90db0f0e7f6716c0e6d0b9cbb67f08d82052b03ab3a7b5e23a1d76aed7 , just moved to the previous case of the switch Co-authored-by: ericLemanissier <[email protected]>
follows-up be4e961240883778c44d9651eaaf9ab8723bbbb0 Fix go-gitea/gitea#33200 --------- Co-authored-by: Giteabot <[email protected]> (cherry picked from commit 57997f1518bada128a6aa775306ec15626cc0691)
Uh oh!
There was an error while loading. Please reload this page.
Description
When mirroring a github repo in gitea (the one causing problem is https://github.com/conan-io/conan-center-index), I noticed the following problems:
the commit graph is unusable, because it shows all the automatic merge of target branch to source branch for PRs existing in the upstream github:

these merge commit also clutter the database (in spite of Mirrors should not migrate
refs/pull
and then the relative commits will be gone. #27961). This can be seen directly in theaction
table in database:It also completely prevents the home page of gitea to load. I get an error 504 by the reverse proxy after 1 minute, and this shows up in the log:
2025/01/10 09:44:30 ...ers/web/user/home.go:129:Dashboard() [E] GetFeeds: FindAndCount: context canceled
.Is there a way to completely avoid mirroring these commits?
Gitea Version
1.22.6
Can you reproduce the bug on the Gitea demo site?
Yes: https://demo.gitea.com/ericLemanissier/conan-center-index and https://demo.gitea.com/ericLemanissier/conan-center-index-git
Log Gist
No response
Screenshots
Git Version
No response
Operating System
No response
How are you running Gitea?
docker
Database
SQLite
The text was updated successfully, but these errors were encountered: