执行摘要

• 一句话：提取 bench 启动/连接决策到可重用 acquire_endpoint
• 推荐动作：建议阅读新模块的设计思路：将“启动 vs 连接”决策封装为单一的 acquire_endpoint 函数，并附带了清晰的关闭生命周期。值得在其他基准测试脚本中复用。该 PR 代码质量高，注释充分，适合作为代码复用和集中决策的范例。

功能与动机

PR body 指出：HTTP 基准测试只需要一个 base URL 发送请求，该 URL 可来自它启动的服务器或已运行的服务器。此前 bench_one_batch_server 内联处理这两种来源的协调，且当设置了 --base-url 时，--host/--port 会被静默忽略。提取后可以消除重复，集中决策，并正确警告冲突参数。

实现拆解

1. 创建新模块：在 python/sglang/benchmark/endpoint.py 中实现了 acquire_endpoint、launch_or_reuse_server、server_is_up、_launch_server_target 以及数据类 BenchEndpoint。acquire_endpoint 是单一入口，根据是否提供 base_url 决定连接已有服务器还是启动新服务器。
2. 修改测试文件：python/sglang/test/bench_one_batch_server_internal.py 删除了原先内联的 launch_server_internal、server_is_up、launch_server_process 函数，改为导入并使用 acquire_endpoint。原来内联的 launch_server_process 逻辑现由 launch_or_reuse_server 替代。
3. 调整入口逻辑：run_benchmark_internal 函数不再直接管理服务器进程，而是调用 acquire_endpoint 获得 BenchEndpoint 实例，并在基准测试结束后调用其 close() 方法（仅当自身启动服务器时才会终止进程）。
4. 配套改动：移除了不再需要的 multiprocessing、os 和 kill_process_tree 导入；新增 acquire_endpoint 导入。

关键文件：

• python/sglang/benchmark/endpoint.py（模块 基准测试模块；类别 source；类型 dependency-wiring；符号 server_is_up, _launch_server_target, launch_or_reuse_server, BenchEndpoint）: 新文件，包含所有集中化的启动/连接逻辑，是 PR 的核心产出。
• python/sglang/test/bench_one_batch_server_internal.py（模块 测试脚本；类别 test；类型 test-coverage；符号 launch_server_internal, server_is_up, launch_server_process）: 修改的测试文件，内联启动逻辑被替换为 acquire_endpoint 调用，验证了新模块的正确性。

关键符号：acquire_endpoint, launch_or_reuse_server, server_is_up, BenchEndpoint.close

评论区精华

此 PR 无 review 评论或实质性讨论，仅有机器人自动回复和作者触发的 rerun-test 命令。

• 暂无高价值评论线程

风险与影响

• 风险：风险极低：核心逻辑原样迁移，未改变服务器启动/健康检查行为。唯一行为变化是当 --base-url 与 --host/--port 同时指定时打印警告，这不会影响模拟正常运行的脚本，但可能对自动化流水线产生预期外的 stderr 输出。若流水线配置为严格检查标准错误，可能需要调整。另外，新模块的导入路径依赖 sglang.benchmark，需确保包结构正确。
• 影响：直接受影响的是 bench_one_batch_server_internal.py 和潜在的未来其他基准测试脚本。开发者现在可以复用 acquire_endpoint 而无需重复实现启动/连接逻辑。对最终用户无影响，仅内部重构。团队需注意新模块的位置，并在后续 benchmark 脚本中推广使用。
• 风险标记：低风险重构, 参数冲突警告可能影响自动化

关联脉络

• 暂无明显关联 PR