Prhub

#45777 [Cleanup] Remove dead env

原始 PR 作者 DarkLight1337 合并时间 2026-06-16 13:56 文件变更 1 提交数 1 评论 0 代码增减 +0 / -3

执行摘要

移除已废弃的 VLLM_RPC_TIMEOUT 环境变量

PR body 明确指出:"Fix a bad merge from https://github.com/vllm-project/vllm/pull/40183"。 PR#40183 引入了 fastsafetensors ParallelLoader,并在重构中移除了 ZMQ RPC 相关逻辑,但环境变量 VLLM_RPC_TIMEOUT 的注册代码被错误保留。本次变更清理了该死代码。

建议合入。这是一次典型的死代码清理,虽然改动微小,但能减少维护负担和认知负荷。对于新贡献者而言,可以学习如何识别并清理合并后残留的配置项。

讨论亮点

无 review 讨论。

实现拆解

仅删除 vllm/envs.py 中的一行环境变量定义和两行注释:

  1. 在环境变量字典中移除键 "VLLM_RPC_TIMEOUT" 及其 lambda 表达式。
  2. 该条目原本设置了 ZMQ 客户端等待后端简单数据操作响应的超时时间(默认 10000 ms)。
  3. 由于 ZMQ 通信路径已被移除,该变量已无任何读取点。
文件 模块 状态 重要度
vllm/envs.py 配置层 modified 4.15

关键源码片段

vllm/envs.py core-logic

环境变量集中注册文件,本次 PR 的唯一修改文件。删除了已废弃的 `VLLM_RPC_TIMEOUT` 配置项。

# vllm/envs.py 中的环境变量字典(仅展示附近上下文)
​
    # 定义环境变量时的典型模式:
    "VLLM_FASTSAFETENSORS_QUEUE_SIZE": lambda: int(
        os.getenv("VLLM_FASTSAFETENSORS_QUEUE_SIZE", "0")
    ),
    # 以下行被删除:
    # "VLLM_RPC_TIMEOUT": lambda: int(os.getenv("VLLM_RPC_TIMEOUT", "10000")),
    # 对应注释也被移除:
    # Time in ms for the zmq client to wait for a response from the backend
    # server for simple data operations
    "VLLM_HTTP_TIMEOUT_KEEP_ALIVE": lambda: int(
        os.environ.get("VLLM_HTTP_TIMEOUT_KEEP_ALIVE", "5")
    ),

评论区精华

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

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

风险与影响

风险极低。VLLM_RPC_TIMEOUT 已被确认不再代码中的任何位置读取(包括 main 分支)。删除操作不会影响运行时行为。唯一可能的影响是如果存在外部脚本或用户显式设置了该环境变量,删除后不再生效,但由于它不再被使用,也不会产生实际后果。

影响范围极小,仅涉及一名开发者及代码清理。无用户可见变更。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论