Skip to content

docs(llmisvc): clarify HTTPRoute spec merge semantics - #702

Open
sahilleth wants to merge 1 commit into
kserve:mainfrom
sahilleth:docs/llmisvc-httproute-spec-merge-semantics
Open

docs(llmisvc): clarify HTTPRoute spec merge semantics#702
sahilleth wants to merge 1 commit into
kserve:mainfrom
sahilleth:docs/llmisvc-httproute-spec-merge-semantics

Conversation

@sahilleth

Copy link
Copy Markdown

What this PR does

Clarifies router.route.http.spec merge behavior in the LLMInferenceService docs so it matches config composition (and kserve/kserve#5827):

  • Top-level fields like hostnames / parentRefs preserve preset rules
  • Timeout-only rules overlay onto preset rules
  • Any other rules list replaces the entire preset Rules list
  • Adds a hostnames example; warns against partial-rules footguns
  • Notes the atomic-list exception in the Strategic Merge Patch section

Related

@netlify

netlify Bot commented Jul 14, 2026

Copy link
Copy Markdown

Deploy Preview for elastic-nobel-0aef7a ready!

Name Link
🔨 Latest commit 1c74f2f
🔍 Latest deploy log https://app.netlify.com/projects/elastic-nobel-0aef7a/deploys/6a762225a1f0e900089967fd
😎 Deploy Preview https://deploy-preview-702--elastic-nobel-0aef7a.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Document that hostnames/parentRefs overlay the router-route preset,
that ruleDefaults overlay per-rule settings onto preset rules, and that
non-empty rules lists replace the entire preset. Addresses kserve/kserve#5706
and aligns with kserve/kserve#5827.

Signed-off-by: Sahil Bhardwaj <sahillbhardwaj23@gmail.com>
@sahilleth
sahilleth force-pushed the docs/llmisvc-httproute-spec-merge-semantics branch from e26162f to 1c74f2f Compare August 7, 2026 18:21
@spolti
spolti requested a review from bartoszmajsak August 7, 2026 19:31
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.

1 participant