-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
GitHub migration: Show "API rate limit of 5000 still exceeded" to user #13242
Labels
topic/repo-migration
Migrate repos from other platforms to Gitea, or from Gitea to them
Comments
7 tasks
A repository notice is always created if there is a failure. gitea/modules/migrations/migrate.go Line 71 in f0fe568
That is of course slightly opaque. |
Ah, but the error should be being set on the task... Line 57 in f0fe568
|
For big repos retry will fail too -- something like sleep+resume migration might work? |
zeripath
added a commit
to zeripath/gitea
that referenced
this issue
Oct 21, 2020
When migrate tasks fail store the error in the task table and ensure that they show on the status page. Fix go-gitea#13242 Signed-off-by: Andrew Thornton <art27@cantab.net>
let's deal with one problem at a time |
6543
added
the
topic/repo-migration
Migrate repos from other platforms to Gitea, or from Gitea to them
label
Oct 23, 2020
techknowlogick
added a commit
that referenced
this issue
Oct 23, 2020
* Store task errors following migrations and display them When migrate tasks fail store the error in the task table and ensure that they show on the status page. Fix #13242 Signed-off-by: Andrew Thornton <art27@cantab.net> * Update web_src/js/index.js * Hide the failed first Signed-off-by: Andrew Thornton <art27@cantab.net> Co-authored-by: techknowlogick <techknowlogick@gitea.io>
techknowlogick
added a commit
to techknowlogick/gitea
that referenced
this issue
Oct 24, 2020
* Store task errors following migrations and display them When migrate tasks fail store the error in the task table and ensure that they show on the status page. Fix go-gitea#13242 Signed-off-by: Andrew Thornton <art27@cantab.net> * Update web_src/js/index.js * Hide the failed first Signed-off-by: Andrew Thornton <art27@cantab.net> Co-authored-by: techknowlogick <techknowlogick@gitea.io>
lunny
pushed a commit
that referenced
this issue
Oct 24, 2020
) * Store task errors following migrations and display them When migrate tasks fail store the error in the task table and ensure that they show on the status page. Fix #13242 Signed-off-by: Andrew Thornton <art27@cantab.net> * Update web_src/js/index.js * Hide the failed first Signed-off-by: Andrew Thornton <art27@cantab.net> Co-authored-by: techknowlogick <techknowlogick@gitea.io> Co-authored-by: zeripath <art27@cantab.net>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
[x]
):Description
When i try to migrate a big GitHub repo, i see that the API limit is reached in the logs, but as an end-user, i see just "migration failed" without reason.
Please show it also to the user and what they can do.
Just wait and try again.
Related to #13241
Log
modules/task/task.go:51:handle() [E] Run task failed: GET https://api.github.com/repos/NixOS/nixpkgs: 403 API rate limit of 5000 still exceeded until 2020-10-21 16:33:28 +0200 CEST, not making remote request. [rate reset in 34m49s]
Screenshots
The text was updated successfully, but these errors were encountered: