Skip to content

[model-gateway] Fix IGW routing for external OpenAI workers#16633

Merged
slin1237 merged 2 commits intosgl-project:mainfrom
zhaowenzi:main
Jan 7, 2026
Merged

[model-gateway] Fix IGW routing for external OpenAI workers#16633
slin1237 merged 2 commits intosgl-project:mainfrom
zhaowenzi:main

Conversation

@zhaowenzi
Copy link
Copy Markdown
Contributor

Motivation

Fix IGW routing where External runtime workers were sent to the regular HTTP router instead of the OpenAI router, so OpenAI-compatible backends received SGLang-specific fields (e.g., priority) and returned “Unknown parameter”.

#16544

Modifications

  • In RouterManager::get_router_for_model, detect RuntimeType::External workers and prefer HTTP_OPENAI, preventing External workers from being handled by the regular router.
  • Keep the existing gRPC/PD/Regular priority ordering unchanged.

Accuracy Tests

N/A

Benchmarking and Profiling

N/A

Checklist

Review Process

  1. Ping Merge Oncalls to start the PR flow. See the PR Merge Process.
  2. Get approvals from CODEOWNERS and other reviewers.
  3. Trigger CI tests with comments (/tag-run-ci-label, /rerun-failed-ci, /tag-and-rerun-ci) or contact authorized users to do so.
  4. After green CI and required approvals, ask Merge Oncalls to merge.

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@zhaowenzi
Copy link
Copy Markdown
Contributor Author

/tag-run-ci-label

@slin1237 slin1237 added the run-ci label Jan 7, 2026
@slin1237 slin1237 merged commit b5a94f8 into sgl-project:main Jan 7, 2026
63 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants