-
Notifications
You must be signed in to change notification settings - Fork 45
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
kr8s and gevent (wsgi) #481
Comments
I think I would agree with the comment by tiangolo fastapi/fastapi#6395 (comment). If That being said I'd be interested to understand more about what is going wrong. If you can share some small examples of how to reproduce the issues you are seeing I can dig in and see if there are any small things we can change to get things to play nicely. Out of interest if creating a sync-native fork is your current workaround why not use a sync-native library like |
That answers it partially 💯 on the other hand my backend's implementation is already founded around kr8s'
I think this is easy to reproduce: setup flask w/ gevent, and the issue happens exactly when invoking
I'm in grief |
Could you share a step by step that I can copy and paste, I'm not familiar with |
I'll be happy to, give me a few days and I'll setup a branch for us 👍 |
Which project are you reporting a bug for?
kr8s
What happened?
When I was just about to celebrate testing a flask app in production.... kr8s took it away from me 🤣
So it turns out it doesn't play well with wsgi. From my analysis it's merely due to the
async
keyword-statements whichgevent
hallucinates on top of, as it runs its own internal monkey patching on python'sthreading
system libraryAs a temporary fix, I'm now running a sync-native fork.
Looking for ideas and brainstorming onto ways we can merge this, if it's the case
Anything else?
fastapi/fastapi#6395
The text was updated successfully, but these errors were encountered: