feat(inference): add compatible local provider failover - #547
Draft
Pal Lakatos-Toth (pallakatos) wants to merge 9 commits into
Draft
feat(inference): add compatible local provider failover#547Pal Lakatos-Toth (pallakatos) wants to merge 9 commits into
Pal Lakatos-Toth (pallakatos) wants to merge 9 commits into
Conversation
Extract complete provider routing, streaming failover, model recovery, and bounded task fallback routes from the canonical integration. Preserve public guardrails and existing-customer schemas, and make local egress opt-in with precise targets. Keep provider credentials router-only and retain final provider/model attribution through Responses recovery. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: eb3654cd-f1e0-445a-8734-430800af1903
Preserve the signing-provider SHA-256 wrappers and known-vector coverage from public/pr3-team-control-plane. Review PR4 public documentation and examples for private deployment identifiers; clarify validation provenance without claiming a post-integration Rust rebuild. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: eb3654cd-f1e0-445a-8734-430800af1903
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
This was referenced Sep 7, 2026
Integrate immutable parent-qualified 415b53c, preserving effective task authority, fail-closed launch budgets, and inference model fallback materialization. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: eb3654cd-f1e0-445a-8734-430800af1903
…play Repair named-provider credential provenance, Copilot account-scoped exchange/cache, authoritative primary routing, typed pre-acceptance failover, account-scoped capability caches, and legacy Helm values. Preserve qualified task authorization and fail-closed launch budgets while verifying model fallback normalization, inheritance, and materialization. Validation: 76 router unit/HTTP tests, 70 controller task/receipt/fallback tests, 6 Helm regressions, and strict all-target Clippy for both crates passed using the existing leased target. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: eb3654cd-f1e0-445a-8734-430800af1903
…sponses Separate legacy primary metadata from explicit native/named routing intent, including Responses recovery and health isolation. Permit truncated-body failover only for known 429/5xx rejections; keep accepted, ambiguous, auth/configuration, and ordinary 4xx failures non-retryable. Add focused HTTP and classifier regressions. Validation: Rust formatting, LOC/crypto gates, and diff checks passed. Rust execution is intentionally deferred until parent assigns the shared Cargo target. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: eb3654cd-f1e0-445a-8734-430800af1903
Integrate immutable 1fd9781, including the current shared authorization snapshot/objective contract, while preserving inference model fallbacks and the bounded PR547 routing repairs. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: eb3654cd-f1e0-445a-8734-430800af1903
Preserve model fallbacks in the shared canonical authorization snapshot and adapt inheritance verification to the current fallible objective API. Record qualified legacy-metadata routing and truncated-rejection recovery on top of immutable 1fd9781. Validation: 84 router tests, 76 controller tests, 6 Helm regressions, strict all-target Clippy for both crates, formatting, LOC/crypto gates, Helm lint and diff checks passed. Existing root target only; no external push. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: eb3654cd-f1e0-445a-8734-430800af1903
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: eb3654cd-f1e0-445a-8734-430800af1903
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: eb3654cd-f1e0-445a-8734-430800af1903
Collaborator
Author
|
Automated closure update: the independent bounded review reports both remaining inference regressions closed at 0a74306 versus c47418f, with no significant regressions found in that repair delta. The subsequent 45cfa00 merge carries only the parent phase-constant and Helm-test timing corrections. Legacy metadata-only primary tags retain default routing unless explicit routing is configured. Truncated known 429/5xx rejections can fail over; accepted responses and authentication/configuration failures cannot. This is automated source review, not human sign-off. Exact-head CI and the capability audit’s genuine author/independent-reviewer signatures remain required; no audit or branch-protection gate is waived. |
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.
Publication stack
This is the next bounded public slice after #543 -> #544 -> #545. It targets the standing-team foundation branch so reviewers see only this inference delta. Do not merge into the intermediate publication branch; after the parent lands, retarget to kars-bridge and require fresh qualification. Base repair commits will be propagated forward.
Scope
No mesh task delivery, runtime orchestration, GitHub writes, access-request service, witness deployment, private H100 values or Bridge code is included. Kars remains independently usable without Bridge.
Qualification evidence
Implementation commit df0540c was locally qualified with 2,119 Rust tests, strict Clippy/formatting, 94 CLI compatibility tests (two skipped), Helm lint and four installation profiles. Three existing Rust doctests were ignored. Local CLI used the available Vitest 4.1.10 cache, not locked 4.1.8; exact-lockfile GitHub CI is required on this integrated head. Foundation hash centralization ea6f578 is incorporated in 7a2a5d1, and six static gates passed afterward.
Repair closure
Provider credentials and health caches are isolated by selected identity; named routes never borrow ambient default credentials. Legacy informational primary-provider metadata preserves the default route. Typed forwarding failures distinguish authentication/configuration, pre-acceptance transport failure, known 429/5xx rejection, and accepted responses. Truncated known rejection bodies remain eligible for fallback, but accepted buffered or streaming generations are never replayed. Canonical authorization, inheritance and materialization retain ordered fallback configuration.
The final repair batch passed 84 router, 76 controller and six Helm regressions, plus strict Clippy for both crates. These local results supplement, rather than replace, the current GitHub checks.
Review gates still open
45cfa009, all technical CI checks pass;security-audit-requiredcorrectly remains blocked on genuine sign-offs. This PR stays draft until the required sign-offs and review gates are satisfied.No main merge, customer deployment, H100 operation or public image release is performed by preparing this PR.
Integration landing plan
The public integration branch
kars-bridgestarts at current main7eb039e9. Land the reviewed, fully qualified slices there in order: #543 -> #544 -> #545 -> #547, followed by the remaining bounded layers. Downstream PRs stay stacked for focused review until their parent lands, then retarget to the integration branch. Preserve merge ancestry; do not squash/restack away reviewed history by default.The integration branch has required CI/security gates, up-to-date-base checks, independent approval and conversation resolution, with force pushes, deletion and admin bypass disabled. Its CI pushes must not automatically publish images or releases. Required genuine audit sign-offs are not waived.
Only after complete cross-layer review and qualification of standalone Kars, existing-customer CLI/install/upgrade behavior, and additive Bridge install/removal should a final integration-to-main promotion PR be merged. No partial runtime is being landed directly in main.