执行摘要
移除死环境变量并修正文档
该变量在 V1 引擎客户端中不再被使用,但文档仍指导用户设置,造成误解。profiling 文档中的相关建议也已不再需要。
此 PR 展示了如何安全清理死代码和同步文档,适用于类似的遗留清理任务。
审核人 sfeng33 指出仍有基准配置文件设置了 VLLM_RPC_TIMEOUT,作者在最后一个提交中清理了所有 CPU 基准配置(共9个文件)。
该变量在 V1 引擎客户端中不再被使用,但文档仍指导用户设置,造成误解。profiling 文档中的相关建议也已不再需要。
此 PR 展示了如何安全清理死代码和同步文档,适用于类似的遗留清理任务。
审核人 sfeng33 指出仍有基准配置文件设置了 VLLM_RPC_TIMEOUT,作者在最后一个提交中清理了所有 CPU 基准配置(共9个文件)。
| 文件 | 模块 | 状态 | 重要度 |
|---|---|---|---|
vllm/envs.py |
配置层 | modified | 5.47 |
docs/contributing/profiling.md |
文档 | modified | 2.22 |
.buildkite/performance-benchmarks/tests/latency-tests-cpu.json |
基准配置 | modified | 2.88 |
.buildkite/performance-benchmarks/tests/serving-tests-cpu.json |
基准配置 | modified | 2.88 |
vllm/envs.py
core-logic
核心变更:删除死环境变量 VLLM_RPC_TIMEOUT 的类型定义和注册条目
# 在 _EnvVars 类中移除了 VLLM_RPC_TIMEOUT 类型定义(原第 98 行)
class _EnvVars:
# ...
VLLM_ALLOW_LONG_MAX_MODEL_LEN: bool = False
# 已移除 : VLLM_RPC_TIMEOUT: int = 10000
VLLM_HTTP_TIMEOUT_KEEP_ALIVE: int = 5 # seconds
# ...
# 在 environment_variables 字典中移除了对应条目(原第 1018-1020 行)
environment_variables = {
# ...
"VLLM_TEST_FORCE_LOAD_FORMAT": lambda: ...,
# 已移除 :
# "VLLM_RPC_TIMEOUT": lambda: int(os.getenv("VLLM_RPC_TIMEOUT", "10000")),
"VLLM_HTTP_TIMEOUT_KEEP_ALIVE": lambda: ...,
# ...
}
sfeng33 在 review 中指出仍有基准配置文件设置了 VLLM_RPC_TIMEOUT,建议一并清理。
结论:作者随后添加提交,从 9 个 CPU 基准配置文件中移除了该变量。 · 已解决
风险极低:该变量已无任何消费者,删除和清理只影响无用的环境变量设置,不影响运行时行为。
用户不再能通过设置 VLLM_RPC_TIMEOUT 影响超时行为(之前也无效);profiling 文档更准确;CI 基准环境变量更简洁。
当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。
参与讨论