Repository navigation
[Automated] Draft docs (agentgateway): llm: respect buffer limits on various paths - #1307
Open
github-actions[bot] wants to merge 1 commit into
Open
github-actions[bot] wants to merge 1 commit into
github-actions[bot] wants to merge 1 commit into
Conversation
…various paths Signed-off-by: GitHub Action <action@github.com>
Deploying agentproxy with
|
| Latest commit: |
baa4c46
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://d06395a5.agentproxy.pages.dev |
| Branch Preview URL: | https://pr-tracker-draft-agentgatewa-xg0o.agentproxy.pages.dev |
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
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.
agentgateway/agentgateway#3896 — llm: respect buffer limits on various paths
Docs versions:
standalone/main,kubernetes/mainWhat changes for users: LLM conversion paths now bound memory growth. Responses namespace tool descriptions fail with
RequestTooLargewhen copied descriptions exceed 8 MiB for one request, and Bedrock Runtime streams return anapi_errorevent withStream processing errorwhen buffered redacted reasoning fragments exceed the stream buffer limit before the block ends.What the docs now say: The standalone Responses page documents the namespace tool description cap. The standalone Messages page and the Kubernetes Bedrock provider page document the streamed Bedrock redacted-reasoning error.
Not verified: No cluster or live Bedrock provider was available; the configuration and stream behavior are unrun.
How
agentgateway-3896was drafted, and what was not verifiedPlan, and what changed it
guardrails[].action, and the candidate guardrails overview pages were not stale.content/docs/standalone/main/documentation/llm/api-types/responses.md, added the Bedrock redacted-reasoning stream error tocontent/docs/standalone/main/documentation/llm/api-types/messages.md, and added the same Bedrock behavior behind a main-only gate inassets/agw-docs/pages/agentgateway/llm/providers/bedrock.md.Why a documentation change is necessary
The pull request changes behavior that a client can observe when large converted LLM streams or namespace tool requests exceed internal limits. The Responses page already documents namespaced tool conversion, and the Messages and Bedrock provider pages already document Bedrock encrypted or redacted reasoning behavior. Without these edits, those pages describe the converted formats but not the new failure modes that stop the request or stream.
What changed on disk
content/docs/standalone/main/documentation/llm/api-types/responses.md: Documents the 8 MiB per-request cap for copied namespace tool descriptions.content/docs/standalone/main/documentation/llm/api-types/messages.md: Documents the Bedrock Runtime stream error for oversized redacted reasoning fragments.assets/agw-docs/pages/agentgateway/llm/providers/bedrock.md: Documents the same Bedrock stream behavior for Kubernetes main and future versions.How to verify this change
RequestTooLarge./v1/messagesrequest to Bedrock Runtime that returns redacted reasoning fragments larger than the stream buffer limit before the content block ends.api_errorstream event with the messageStream processing error, and the stream stops.What was not verified
How the pages were chosen (triage report)
New configuration: no page mentions it yet
agentgateway/agentgateway#3896 — llm: respect buffer limits on various paths
agentgateway/websiteToolBlock.argumentsbuffer that kept every tool-call argument delta for the whole stream. - completions (Messages->Completions): only collect streamed tool-call arguments when tool-call logging is enabled. - bedrock (Converse->Messages): cap buffered redacted-reasoning bytes atbuffer_limitand send a stream error when it's exceeded; fragmRelease notes
This pull request also adds its release-note entries:
content/docs/kubernetes/main/release-notes/release-notes.md: llm: respect buffer limits on various paths (fix, under LLM)content/docs/standalone/main/release-notes/release-notes.md: llm: respect buffer limits on various paths (fix, under LLM)Check these release-note links and names. Each note is on its page, but a link may not reach the drafted content there, or a name may belong to the other mode:
content/docs/kubernetes/main/release-notes/release-notes.md: llm: respect buffer limits on various paths agentgateway#3896's note namesStream processing error, whichcontent/docs/standalone/mainmentions andcontent/docs/kubernetes/maindoes not. Check that this tree's readers can set it, or word the note for this mode.content/docs/kubernetes/main/release-notes/release-notes.md: llm: respect buffer limits on various paths agentgateway#3896's note namesRequestTooLarge, whichcontent/docs/standalone/mainmentions andcontent/docs/kubernetes/maindoes not. Check that this tree's readers can set it, or word the note for this mode.Draft branch:
pr-tracker-draft-agentgateway-3896