-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Wikidata calls failing, likely due to proxy limitations / rate limits #10444
Comments
@mekarpeles How many total call are we making vs how many are failing? |
Are any of these calls succeeding? I think that we need to update the endpoint: |
@jimchamp I think you're exactly right. As I understand this endpoint doesn't have a rate limit and if it did it should be giving a header to state such. So we should update the url. |
We may also want to record non- |
I've opened #10452 to update the endpoint. |
#10452 has been patch deployed, and I can see that entries are again being added to the |
#10456 adds Wikidata API errors to our Sentry logs. I'm still seeing new |
Add the 20 errors still 503s? Is it possible to get the IDs of the entities failing? |
They're all the same: Looks like these errors from the past day have largely been triggered by bots: The IDs are included in the event title (pictured here). Do you want a small sample, or all of the events that occurred since the patch deployment? |
Problem
https://sentry.archive.org/organizations/ia-ux/issues/540536/?environment=production&p[…]kidata&referrer=issue-stream&statsPeriod=12h&stream_index=0
The first line works, attempts to connect to ipv4, the second one fails:
Hypothesis
The hypothesis is that, since we implemented proxy requests, we are no longer making requests from 3 servers (ol-web[0-2]) and instead just one. We need to explore a way to either cache more aggressively (for longer) or batch lookups in the same way that our bookworm affiliate-server does. Otherwise, the recommendation is we temporarily disable the calls to avoid making too many calls to wikidata.
We are hitting several thousand errors per 12h time period.
cc: @RayBB @cdrini
Reproducing the bug
Context
Breakdown
Requirements Checklist
Related files
Stakeholders
Instructions for Contributors
The text was updated successfully, but these errors were encountered: