Skip to content

Migrate GitHub ingestion to webhooks #87

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.

Sign up for GitHub

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

Jump to bottom
Closed
anp opened this issue Oct 7, 2016 · 2 comments
Closed

Migrate GitHub ingestion to webhooks #87

anp opened this issue Oct 7, 2016 · 2 comments
Labels
rfcbot
Milestone
Adam goes incognito

Comments

@anp
Copy link
Member

anp commented Oct 7, 2016 •
edited
Loading

There have been several examples over the last week of increased usage which have seen long delays for rfcbot response times (as much as 30 min). I'm assuming this is closely related to the polling-based architecture I'm using to be able to cover so many repositories without configuration, but it's time to dig into the logs as to why the times are half an hour, instead of 3 minutes (which is the delay it's configured for).

See #61 (comment) for an example (not 30 minutes, but still...).
@anp anp added the rfcbot label Oct 7, 2016
@anp
Copy link
Member Author

anp commented Oct 17, 2016

Note to self: this is definitely due to API limits. Perhaps time to switch to webhooks?

@anp anp changed the title rfcbot response time is terrible Migrate GitHub ingestion to webhooks Oct 17, 2016
@anp anp added this to the Adam goes incognito milestone Oct 22, 2016
@anp
Copy link
Member Author

anp commented May 1, 2017

This is done!

@anp anp closed this as completed May 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
rfcbot
Projects
None yet
Milestone
Adam goes incognito
Development

No branches or pull requests
1 participant
@anp