Skip to content

Add litellm-proxy CLI #10478

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

Open
wants to merge 74 commits into
base: main
Choose a base branch
from
Open

Add litellm-proxy CLI #10478

wants to merge 74 commits into from

Conversation

msabramo
Copy link
Contributor

@msabramo msabramo commented May 1, 2025

Title

Add litellm-proxy CLI

$ litellm-proxy
Usage: litellm-proxy [OPTIONS] COMMAND [ARGS]...

  LiteLLM Proxy CLI - Manage your LiteLLM proxy server

Options:
  --base-url TEXT  Base URL of the LiteLLM proxy server  [env var:
                   LITELLM_PROXY_URL]
  --api-key TEXT   API key for authentication  [env var:
                   LITELLM_PROXY_API_KEY]
  --help           Show this message and exit.

Commands:
  chat         Chat with models through the LiteLLM proxy server
  credentials  Manage credentials for the LiteLLM proxy server
  http         Make HTTP requests to the LiteLLM proxy server
  keys         Manage API keys for the LiteLLM proxy server
  models       Manage models on your LiteLLM proxy server

Right now this only has a limited set of commands, but in the future, more capabilities can be added.

More details at the README.md here

Screenshots

Screenshot 2025-05-01 at 7 01 54 AM Screenshot 2025-05-01 at 7 04 18 AM

Relevant issues

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

  • I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • I have added a screenshot of my new test passing locally
  • My PR passes all unit tests on make test-unit
  • My PR's scope is as isolated as possible, it only solves 1 specific problem

Type

🆕 New Feature

Changes

msabramo added 30 commits April 29, 2025 21:38
@msabramo
Copy link
Contributor Author

msabramo commented May 2, 2025

What happened here? The tests passed but the job failed. Is the job timing out perhaps?

Screenshot 2025-05-02 at 10 51 19 AM

I wonder if #10484 would fix this...

because it looks like maybe the job is getting canceled because it takes
too long with the additional tests?

This probably could be reverted once BerriAI#10484 is merged, since that speeds
up pytest runs greatly.
@msabramo
Copy link
Contributor Author

msabramo commented May 2, 2025

What happened here? The tests passed but the job failed. Is the job timing out perhaps?

Screenshot 2025-05-02 at 10 51 19 AM

I wonder if #10484 would fix this...

OK, I fixed this I think with ca39496 (Increase test job timeout to 8 minutes), though I think it would be preferable to merge #10484 and then this timeout increase wouldn't be necessary

@krrishdholakia
Copy link
Contributor

Hey @msabramo let me know once this is ready for review

@msabramo
Copy link
Contributor Author

msabramo commented May 3, 2025

Oh this is ready for review. I added a few things to it recently only because I thought you were busy with fixing the mypy issues, but go ahead and review now if you have the time!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants