执行摘要

• 一句话：移除 rollout 中已废弃的指标地址方法
• 推荐动作：简单清理，可快速合并。建议后续关注是否有其他未使用的公共接口，可系统性清理。

功能与动机

PR body 和标题为 "cleanup"，结合变更内容，这两个方法已在之前 PR（如 #2070 和 #2036）中不再被调用，属于死代码清理。清理可以提高代码可维护性，减少混淆。

实现拆解

在 slime/ray/rollout.py 中，删除如下两个方法：

1. _get_metrics_router_addr(self) -> str | None：返回第一个服务器的路由器地址以供抓取 SGLang Prometheus 指标。
2. get_metrics_router_addr(self) -> str | None：公共封装方法，用于从 driver 进程远程调用。

由于这两个方法在其他地方已无引用，直接移除不影响现有逻辑。

关键文件：

• slime/ray/rollout.py（模块 Rollout；类别 source；类型 core-logic；符号 _get_metrics_router_addr, get_metrics_router_addr）: 删除两个未使用的方法，减少核心 Rollout 类的公共接口，降低维护成本。

关键符号：_get_metrics_router_addr, get_metrics_router_addr

关键源码片段

slime/ray/rollout.py

删除两个未使用的方法，减少核心 Rollout 类的公共接口，降低维护成本。

# 以下两个方法在 PR 中被删除，因为它们已不再被调用
​
# def _get_metrics_router_addr(self) -> str | None:
# """Return the router address for scraping SGLang engine metrics.
#
# The sglang_router gateway exposes ``/engine_metrics`` on its main port,
# which aggregates Prometheus metrics from all backend sglang servers.
# Returns ``http://{ip}:{port}`` for the first server, or ``None`` when
# metrics are disabled or no servers are running.
# """
# srv = self.server
# if srv is None or srv.router_ip is None:
# return None
# return f"http://{srv.router_ip}:{srv.router_port}"
#
# def get_metrics_router_addr(self) -> str | None:
# """Public wrapper for remote calls from the driver process."""
# return self._get_metrics_router_addr()

评论区精华

该 PR 无 review 评论。

• 暂无高价值评论线程

风险与影响

• 风险：风险极低。仅删除两个未使用的方法，未修改其他逻辑。但需确认是否有外部依赖（如脚本或工具）通过反射或动态调用依赖这些方法，但当前仓库无此迹象。
• 影响：影响范围小。仅影响开发者或外部工具可能通过公共 API 调用 get_metrics_router_addr 的场景。核心训练/推理流程不受影响。
• 风险标记：暂无

关联脉络

• 暂无明显关联 PR