search

kernelci / kernelci-pipeline

Modular pipeline based on the KernelCI API
GNU Lesser General Public License v2.1
8 stars 21 forks source link

fix(lava_callback.py): Fix crash of lava_callback.py #871

Closed nuclearcat closed 2 weeks ago

nuclearcat commented 2 weeks ago

Stacktrace observed in production: File "/usr/local/lib/python3.11/site-packages/fastapi/routing.py", line 191, in run_endpoint_function return await dependant.call(**values) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/home/kernelci/src/lava_callback.py", line 280, in callback data = await request.json() ^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.11/site-packages/starlette/requests.py", line 251, in json body = await self.body() ^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.11/site-packages/starlette/requests.py", line 244, in body async for chunk in self.stream(): File "/usr/local/lib/python3.11/site-packages/starlette/requests.py", line 238, in stream raise ClientDisconnect() starlette.requests.ClientDisconnect

This should fix it and return error properly.

nuclearcat commented 2 weeks ago

Tested on staging