-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
📝 add deployment docs with Granian #9750
Conversation
📝 Docs preview for commit 4467a44 at: https://6499e4188604e005d54425c6--fastapi.netlify.app |
Any news on this? |
FYI Granian 1.0 added support for lifespan state, thus the |
@tiangolo @gi0baro It would be nice to add |
📝 Docs preview for commit c3e4adb at: https://f18da17c.fastapitiangolo.pages.dev |
@tiangolo In which release it will be available? |
1 similar comment
@tiangolo In which release it will be available? |
@tiangolo any update? |
tl;dr I'd really like to have an update from a maintainer here to – finally – have this merged or rejected. I feel quite uncomfortable making this comment, and I've spent a few hours thinking about this, but given this is a 39 lines docs diff opened more than 1 year ago, here I am. I don't know what the p3 label sticked on this for 6 months means, but it doesn't feel right to me leaving this kind of PRs in this limbo without any form of feedback/change request/comment. And again, I never claimed to have this merged; this can be easily rejected, as soon as there's a reason for that decision. The total absence of communication, on the other hand, doesn't really seem a good process, at least to me. I never wanted to unsubscribe or close this, but the frustration is tempting me in this direction, sadly. |
Thanks for the comment @gi0baro, I understand your frustration. About this PRHere's the thing, I would need to see where are the places where notes around this would be relevant. It would probably make sense to include several alternatives, e.g. Nginx Unit. But then I'm not sure it would make sense to have instructions for each of them, or maybe it would be better to have a page listing the servers with links to their websites. I'm also thinking of simplifying the docs not having too many options everywhere, to facilitate the decision, that was one of the reasons to build the So, it's not about merging or rejecting the 39 lines, but about reviewing all the rest of the docs and places where it would make sense to have it or not. About the process and frustrationNow, even though I would probably be included to simplify all the docs and not have execution instructions for each tool and instead have a page listing several or all the available ASGI servers, I'm not decided on that yet. I haven't sat down to review this PR and review all the relevant sections and information. So, I'm not decided on this PR, about rejecting it or (probably tweaking it) and accepting it. My thought was that closing it saying "I'm not sure yet", wouldn't be nice. So I was defaulting to leaving it open until I decided to take it or when I had made an alternative PR that would supersede it. |
Thank you for the reply, I appreciate.
I totally understand that @tiangolo; again, I slipped into the same condition on my projects. But that's exactly where the frustration comes from (and came to others when I was on the other side of the story), 'cause when I read this
everything makes perfect sense.
Letting me now of that when you first felt unsure might have led to keep the discussion open. Would you be interested in a PR that tackles servers and deployment in a more dedicated way, and moving the discussion over there? |
Thanks for the feedback @gi0baro! I took that in mind while setting up the new structure for us to handle new PRs, so there's clearer feedback. Thanks! I think it will be easier for me to write the new section about deployment myself than reviewing an external PR without the ideas I've had in mind. So, probably better let's keep this on stand by for a bit. Thanks! |
I understand the want to refactor, but I do keep coming back to this PR (from google) to find that launch line for granian in FastAPI. Merging this simple change, and then refactoring documentation would be the more efficient route (for me personally), and might also clear up your mental backlog. |
Thanks @gi0baro and everyone for the interest here! 🍰 I just updated the docs for serving FastAPI apps here: #12069 There are now links to several ASGI servers, including Granian. 🚀 I updated the other parts to simplify and reduce the options and recommendations, so, the docs in FastAPI are now simpler, but with links to several alternatives so that everyone can explore more on their own. Given that, I'll now close this one. Thanks! ☕ |
Granian is a Rust HTTP server for Python applications which supports ASGI and HTTP/2.
It isn't as well known in the FastAPI community as an option - hopefully this will help that.