-
Notifications
You must be signed in to change notification settings - Fork 15
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
Add Jitter to "Update Go" action schedule #979
Add Jitter to "Update Go" action schedule #979
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you very much for this. This was a high priority fix.
Two very minor notes around doc and I think this should be good to merge.
55b1f52
to
aa78e4d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Thanks very much!
Summary
Fixes #966 by adding a Jitter to the cron schedule (Alternative 2 in the issue)
The jitter is seeded with the repo name (
GITHUB_REPOSITORY
env) and will there produce a constant value for the same repo (unless renamed).It is implemented in a separate package to be easily reused in other actions that might profit from this as well.
Use Cases
Prevent rate limiting
Checklist