PR 说明指出：5 个 VLLM_* 环境变量在代码树中已无任何消费者，且 EP 部署文档仍在指导用户调整一个已被删除的变量。清理这些死代码和过期引用，使环境变量表面与文档匹配，减少用户困惑。
推荐直接合并。该 PR 是常规维护清理，无技术异议，已获 reviewer 批准。
主要讨论集中在 PR 的等待过程（约 6 天无审查）和一次 rebase 事件。作者在评论中解释了 rebase 原因：分支曾漂移到错误 commit（误删了仍有消费者的变量），导致 CI 失败。回滚后确认原描述的 5 个变量在最新 main 上仍无消费者。
vllm/envs.py 中的类型定义：删除 VLLM_ROCM_USE_AITER_PAGED_ATTN、VLLM_TPU_BUCKET_PADDING_GAP、VLLM_TPU_MOST_MODEL_LEN、VLLM_CI_USE_S3、VLLM_FLASHINFER_ALLREDUCE_FUSION_THRESHOLDS_MB 的类型注解行（共 5 行）。vllm/envs.py 中的运行时取值逻辑：删除上述 5 个变量的 lambda 取值器（每个对应一段 os.getenv 调用），并清理 compile_factors() 的 ignored_factors 集合中对 VLLM_CI_USE_S3 的引用。docs/serving/expert_parallel_deployment.md 中过期建议：移除关于调整 VLLM_MOE_DP_CHUNK_SIZE 来提高吞吐的段落（该变量已在 #39107 重构中移除）。docs/configuration/env_vars.md 的表述：将“所有环境变量以 VLLM_ 为前缀”改为“大部分 vLLM 特定环境变量以 VLLM_ 为前缀”，并举例说明 CUDA_VISIBLE_DEVICES、MAX_JOBS 等标准变量同样被直接读取。| 文件 | 模块 | 状态 | 重要度 |
|---|---|---|---|
vllm/envs.py |
核心配置 | modified | 6.4 |
docs/serving/expert_parallel_deployment.md |
部署文档 | modified | 2.1 |
docs/configuration/env_vars.md |
配置文档 | modified | 1.54 |
vllm/envs.py
core-logic
删除 5 个已无消费者的环境变量定义及其运行时取值逻辑，是 PR 的核心变更文件。
# 从 EnvVars 类定义中删除的行（示例）：
# - VLLM_ROCM_USE_AITER_PAGED_ATTN: bool = False # 已被 #30586 删除消费者
# - VLLM_TPU_BUCKET_PADDING_GAP: int = 0 # 已被 #30808 迁移
# - VLLM_TPU_MOST_MODEL_LEN: int | None = None # 同上
# - VLLM_CI_USE_S3: ... # 无消费者
# - VLLM_FLASHINFER_ALLREDUCE_FUSION_THRESHOLDS_MB: ... # 无消费者
# 同时删除对应的 lambda 取值器，例如：
# "VLLM_ROCM_USE_AITER_PAGED_ATTN": lambda: (
# os.getenv("VLLM_ROCM_USE_AITER_PAGED_ATTN", "False").lower() in ("true", "1")
# ),
# 并从 compile_factors() 的 ignored_factors 集合中移除 'VLLM_CI_USE_S3'
作者在评论区多次催促审查，并在合并冲突后进行了 rebase。值得注意的是，分支曾漂移到错误 commit，误删了仍有消费者的变量（如 VLLM_BUILD_COMMIT），导致 CI 失败。
结论：作者回滚至正确 commit，确认删除集合无误。 · 已解决
风险极低，因为删除的变量均由作者通过 grep -rl 在全树（排除 vllm/envs.py）验证无消费者。但需注意：如果某个模块后来通过字符串拼接或间接方式引用这些变量（例如动态构造变量名），则可能存在遗漏。不过从代码结构看，这些变量均为直接引用，风险几乎可忽略。
用户影响：用户无法再通过设置这些环境变量调整行为（但实际已无效果）。系统影响：减少 vllm/envs.py 的维护负担和文档误导。团队影响：小规模清理，无需回归测试。
当前没有检测到明确关联的 Issue 链接，后续同步到相关引用后会出现在这里。
参与讨论