-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
[TODO] Figure out the upstream issue causing ClientPayloadError: Response payload is not completed
#59
Comments
@felixfontein I saw you pinged me somewhere on Matrix, but I lost the notification/room so mentioning you here. This is essentially what's been happening to Patchback lately. |
This https://stackoverflow.com/a/70172596/595220 suggests that the problem might only manifest itself if Python is higher, than 3.8. |
Though, the upstream bug that also shows gidgethub+aiohttp claims that it's present on Python 3.8 too: aio-libs/aiohttp#4581. |
UPD: with sanitizers/patchback-github-app#41 (comment), we should be getting more details soon. |
The currently deployed bots have been experiencing this problem for PATCH and GET requests:
It's been happening since Oct 31 with various levels of intensity. Chronographer only experienced this 3 times all within a 32-second interval, on Nov 16. Patchback has 1-2 errors like this per day.
The bots moved from OpenShift 3 to a new OpenShift 4-based environment on Oct 16, so there is a chance that the CPython runtime environment got updated as well. The python-level deps are pinned via pip-tools. Both have
aiohttp==3.6.2
,multidict==4.7.6
, andyarl==1.5.1
. But it's hard to know if the CPython version has changed since I don't have logs and the old OpenShift instance is no longer available. Sentry reports suggest that both bots are running under CPython 3.9.13. The pre-OpenShift configs that used to be used in Heroku have CPython 3.8.6 in them, so maybe old OpenShift was running that.Also, there are no Sentry reports that would suggest the same problem happening within the old OpenShift (the history goes 90 days back). This probably means that the Fedora container was different and maybe, the CPython version was too.
Upvote & Fund
The text was updated successfully, but these errors were encountered: