# PR #2080 完整报告

- 仓库：`THUDM/slime`
- 标题：cleanup
- 合并时间：2026-06-15 11:41
- 原文链接：http://prhub.com.cn/THUDM/slime/pull/2080

---

# 执行摘要

- 一句话：移除 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 类的公共接口，降低维护成本。

```python
# 以下两个方法在 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