docs(deploy): show ingress.extraPaths on the componentized helm chart - #745
Open
yassin-berriai wants to merge 1 commit into
Open
docs(deploy): show ingress.extraPaths on the componentized helm chart#745yassin-berriai wants to merge 1 commit into
yassin-berriai wants to merge 1 commit into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Open
5 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Companion to BerriAI/litellm#35700, which adds
ingress.extraPathsto the componentized helm chartThe componentized chart's Ingress renders a fixed path set, and the prefix list it routes to the gateway is a snapshot of the data plane at chart release time. A passthrough route the list omits falls into the backend catch-all and 404s, and a custom
general_settings.pass_through_endpointspath has a name only the operator knows, so the chart can never ship a rule for it.ingress.extraPathslets an operator add those paths fromvalues.yamlwithout forking the template.The helm tab's ingress example now shows the knob with its two selectors and their defaults, next to the values it already demonstrates
Merge after BerriAI/litellm#35700, since the knob does not exist until that lands
Validated with
npm ci && npm run build; the only warnings are the pre-existing broken anchors on old release-notes pages