-
-
Notifications
You must be signed in to change notification settings - Fork 3.8k
[Feat] LiteLLM Allow setting Uvicorn Keep Alive Timeout #11594
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds support for configuring the Uvicorn keepalive timeout in LiteLLM, enhancing the server configuration flexibility. Key changes include adding a new CLI option and associated parameter in the Uvicorn initialization helper, updating tests for the new parameter, and documenting the feature in deployment instructions.
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
File | Description |
---|---|
tests/test_litellm/proxy/test_proxy_cli.py | Added tests to verify the keepalive_timeout flag and parameter usage. |
litellm/proxy/proxy_cli.py | Introduced the keepalive_timeout parameter in CLI and helper methods. |
docs/my-website/docs/proxy/deploy.md | Updated documentation with usage examples for keepalive_timeout. |
Comments suppressed due to low confidence (1)
docs/my-website/docs/proxy/deploy.md:688
- The documentation specifies a default keepalive timeout of 5 seconds, but the CLI option default is None and no default value is set in the helper function. Please update the code to set a default of 5 seconds or modify the docs to reflect the current behavior.
Defaults to 5 seconds. Between requests, connections must receive new data within this period or be disconnected.
* Add keepalive timeout option for uvicorn server configuration * docs Keepalive Timeout --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com>
* Add keepalive timeout option for uvicorn server configuration * docs Keepalive Timeout --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com>
[Feat] LiteLLM Allow setting Uvicorn Keep Alive Timeout
This PR adds support for configuring the Uvicorn keepalive timeout in LiteLLM
Relevant issues
Pre-Submission checklist
Please complete all items before asking a LiteLLM maintainer to review your PR
tests/litellm/
directory, Adding at least 1 test is a hard requirement - see detailsmake test-unit
Type
🆕 New Feature
✅ Test
Changes