sanitizers / octomachinery

🤖 Invisible engine driving octobot machines. Simple, yet powerful. [DEMO BOT @ https://github.com/sanitizers/chronographer-github-app] | [tutorial @ https://tutorial.octomachinery.dev] | [docs @ https://docs.octomachinery.dev] | official web-site is at -->
https://octomachinery.dev
GNU General Public License v3.0
56 stars 12 forks source link

[TODO] Figure out the upstream issue causing `ClientPayloadError: Response payload is not completed` #59

Closed webknjaz closed 3 weeks ago

webknjaz commented 9 months ago

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, and yarl==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

Fund with Polar

webknjaz commented 9 months ago

@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.

webknjaz commented 9 months ago

This https://stackoverflow.com/a/70172596/595220 suggests that the problem might only manifest itself if Python is higher, than 3.8.

webknjaz commented 9 months ago

Though, the upstream bug that also shows gidgethub+aiohttp claims that it's present on Python 3.8 too: https://github.com/aio-libs/aiohttp/issues/4581.

webknjaz commented 7 months ago

UPD: with https://github.com/sanitizers/patchback-github-app/issues/41#issuecomment-1948866825, we should be getting more details soon.

webknjaz commented 3 weeks ago

This seems to have been fixed upstream, in aiohttp v3.10.5.