-
Notifications
You must be signed in to change notification settings - Fork 4
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
Error: You have exceeded a secondary rate limit. Please wait a few minutes before you try again. #39
Comments
It's a good idea. Maybe there's built in support for rate limit / retry in https://github.com/actions/toolkit? |
I dont think that toolkit is being maintained much anymore. Even basic commands like Curl have built in support now days. .e.g curl --retry 5 https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Retry-After |
I see commits a few days ago. Either way I'd start there and implement a retry in a way that's a bit lower level than create-a-github-issue, then we can always include that code here if needed. |
Great, look forward to testing it out. |
Looking forward to your PR! |
Was there any progress on this issue? |
Haven't heard from @SteveBox0. AFAIK nobody is working on this. |
Hey I have just noticed one more error related to this topic
Even If the run fails with limit errors, this does not stop from the issue creation, ideally if the step fails with error @dblock @SteveBox0 have you noticed the same issue ? Thanks |
@prudhvigodithi I'm pretty sure that the issue is not created, your example above shows 2.9.0 while this screenshot is 2.6.0 (different numbers)? This issue is a feature request to retry. |
Hey @dblock, same with Error:
|
@prudhvigodithi it would be a separate bug - this one is a feature request for retrying, so please open another one if you see something not working |
Thanks will create a new issue related to this topic. |
As per Github docs it should retry after a period of time based off the HTTP response header "Retry-After: ".
https://docs.github.com/en/rest/guides/best-practices-for-integrators?apiVersion=2022-11-28#dealing-with-secondary-rate-limits
The text was updated successfully, but these errors were encountered: