Skip to content

Conversation

@LukasdeBoer
Copy link
Contributor

@LukasdeBoer LukasdeBoer commented Dec 5, 2025

Title

Add possibility to override command, args and add deployment labels in the Helm Chart. This is something we need to support injecting secrets at runtime using the Vault sidecar, to load secrets into ENV and then starting up litellm proxy.

Pre-Submission checklist

  • 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
image

Type

🆕 New Feature
✅ Test

Changes

This PR adds three new configuration options to the Helm chart for greater flexibility in deployment customization:

New Values:

  • command - Override the container command (entrypoint). Useful for custom initialization scripts or debugging.
  • args - Override the default container arguments. By default, the chart passes --config /etc/litellm/config.yaml (and optionally --num_workers), but this can now be fully customized.
  • deploymentLabels - Add custom labels to the Deployment metadata. Useful for organization-specific labeling conventions, compliance requirements, or tooling integration.

@vercel
Copy link

vercel bot commented Dec 5, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
litellm Ready Ready Preview Comment Dec 5, 2025 7:58pm

@CLAassistant
Copy link

CLAassistant commented Dec 5, 2025

CLA assistant check
All committers have signed the CLA.

@krrishdholakia
Copy link
Contributor

@LukasdeBoer can you fix the helm linting errors please?

@LukasdeBoer
Copy link
Contributor Author

Hey @krrishdholakia, helm issues should be fixed now, both the linting and unit test!

@krrishdholakia krrishdholakia merged commit 3b8a6ec into BerriAI:main Dec 6, 2025
4 of 7 checks passed
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