Prhub

#2107 Fix wrongly removed RouterArgs registry

原始 PR 作者 zhuzilin 合并时间 2026-06-20 22:58 文件变更 1 提交数 1 评论 0 代码增减 +2 / -0

执行摘要

修复 SGLang RouterArgs 注册遗漏

SGLang 升级后,重构过程中误删了 RouterArgs.add_cli_args 调用,导致 SGLang 路由器的标准参数无法通过 --sglang-router-* 前缀自动注册,破坏了原有配置接口的一致性。PR body 未详细说明,但从提交信息「Fix wrongly removed RouterArgs registry」和仅 2 行新增的变更内容可明确判断这是一个回归修复。

变更微小且修复明确,可快速合入。建议开发者检查 sglang_router 库版本是否为预期版本(参考 Dockerfile 或 sglang 依赖),避免因库接口变化引入新的不兼容。

讨论亮点

该 PR 无 review 评论。

实现拆解

  1. 在函数 add_sglang_router_arguments 末尾新增一行 RouterArgs.add_cli_args(parser, use_router_prefix=True, exclude_host_port=True),调用 sglang_router 库提供的 CLI 参数注册方法,将所有 RouterArgs 支持的参数以 --sglang-router- 前缀自动注入 parser。
  2. 传参 use_router_prefix=True 确保参数均带有 --sglang-router- 前缀,避免与全局参数冲突;exclude_host_port=True 排除 hostport 等已在别处管理的参数,保持功能与 SGLang 官方用法一致。
  3. 无测试、配置或部署配套改动。
文件 模块 状态 重要度
slime/backends/sglang_utils/arguments.py SGLang 参数 modified 4.82

关键符号

add_sglang_router_arguments

关键源码片段

slime/backends/sglang_utils/arguments.py dependency-wiring

唯一变更文件。恢复被误删的 `RouterArgs.add_cli_args` 调用,修复 SGLang 路由器参数注册遗漏。

import argparsefrom sglang.srt.server_args import ServerArgs
from sglang_router.launch_router import RouterArgs # 补回 RouterArgs 导入
from slime.utils.http_utils import _wrap_ipv6
​
​
def add_sglang_router_arguments(parser):
    parser.add_argument("--sglang-router-ip", type=str, default=None)
    parser.add_argument("--sglang-router-port", type=int, default=None)
    parser.add_argument("--sglang-router-request-timeout-secs", type=int, default=14400)
    # 补回该调用:自动注册所有 RouterArgs 支持的参数,均带 --sglang-router- 前缀
    # exclude_host_port=True 避免与手动管理的 host/port 冲突
    RouterArgs.add_cli_args(parser, use_router_prefix=True, exclude_host_port=True)
    return parser

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。

风险与影响

变更极微(仅 2 行新增),风险极低。但需要注意 RouterArgs.add_cli_args 的接口是否随 sglang_router 库升级发生不兼容变化(例如参数名、前缀行为),建议确保 sglang_router 版本与本次回归的上下文一致。

仅影响 SGLang 路由器参数解析流程,影响范围局限于使用 SGLang 路由器的训练/推理任务。对不使用 SGLang Router 功能的用户无影响。

缺少测试覆盖

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论