fix(stargate): bound proxy retries and relay retention - #1820
Draft
barrygreengus wants to merge 1 commit into
Draft
fix(stargate): bound proxy retries and relay retention#1820barrygreengus wants to merge 1 commit into
barrygreengus wants to merge 1 commit into
Conversation
barrygreengus
added this pull request to stack #1821
September 11, 2026 21:05
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueComment |
Contributor
🛡️ CodeQL Analysis🚨 Found 5 issue(s) Severity Breakdown:
📋 Top Issues🔗 View full details in Security tab 🕐 Last updated: 2026-09-11 21:22:52 UTC | Commit: 2d1ee42 |
This was referenced Sep 11, 2026
Stop replaying submitted POST requests after ambiguous transport failures, drain completed relay tasks, and aggregate unknown routing targets under bounded metric labels. Refs: #1817
barrygreengus
force-pushed
the
codex/stargate-maintenance/proxy
branch
from
September 11, 2026 22:10
2d1ee42 to
344a6e4
Compare
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.
Why
A response-header timeout could replay a POST after the backend had accepted it. Long-lived QUIC relays retained completed tasks, and arbitrary unknown model/routing-key headers created permanent metric series.
What changed
Customer Release Notes
Stargate avoids duplicate inference requests after uncertain transport failures and bounds memory retained by relay completions and unknown-target metrics.
Plan Summary
Not applicable.
Usage
A transport failure after body submission returns the upstream transport error to the caller. Explicit retryable pylon rejection behavior is preserved.
Testing
The rebased stack tip passed all 1,611 Cargo workspace tests, including 143 Stargate integration tests, with 3 existing ignored performance tests. Workspace Clippy passed with warnings denied. The POST regression confirms one backend execution and a 502 response after a header timeout. Relay lifecycle and bounded unknown-target metric regressions pass. No live-cluster or production traffic soak was run.
Notes
Layer 2 of the Stargate maintenance stack. Unknown-target 404 metrics use empty routing_key, model, and inference_server_id labels; raw rejected identities remain in logs.
Issues
Relates to #1817
References
HTTP retry semantics
Related Pull Requests
Depends on #1818. Next layer: #1822.
Dependencies
No added or upgraded dependencies. No license or NOTICE changes.