papr icon indicating copy to clipboard operation
papr copied to clipboard

github api rate limit exhaustion

Open cgwalters opened this issue 7 years ago • 2 comments
trafficstars

We've been hitting this periodically.

We may need to dive into figuring out what's actually using the key; offhand guess is homu's "check all outstanding PRs for mergability after a merge" is part of the problem

Also https://developer.github.com/v3/#conditional-requests

homu probably isn't doing that, and i wouldn't be surprised if jenkins isn't either

cgwalters avatar Jan 12 '18 21:01 cgwalters

One snag is that AFAICT GitHub doesn't provide any way to visualize what requests were made, which would've been a nice debugging tool.

So the three users of the token are:

  • GHPRB
  • PAPR
  • Homu

GHPRB is probably a big consumer too, yeah. Without being able to use webhooks, it needs to be able to rescan PRs when polling to look for changes. AFAICT, the GitHub API library it uses doesn't make use of conditional requests.

jlebon avatar Jan 12 '18 21:01 jlebon

We're hitting this every hour right now. As a short-term workaround, I've submitted a request to drop testing for openshift-ansible: https://github.com/openshift/release/pull/550.

jlebon avatar Jan 15 '18 17:01 jlebon