执行摘要
- 一句话:移除 Qwen-7B-Chat 相关测试条目
- 推荐动作:该 PR 是清理性变更,技术价值低,无需精读。可关注关联 PR #45131 了解模型移除的详细动机。
功能与动机
PR Body 指出:"Fixes https://buildkite.com/vllm/ci/builds/71234#019eb2da-8fd7-4096-b373-2593fb49305b Qwen has been removed in https://github.com/vllm-project/vllm/pull/45131, so we remove the related tests as well." 即模型 Qwen/Qwen-7B-Chat 已在 #45131 中被移除,但测试配置仍引用了该模型,导致 CI 构建失败(报错 ValueError: No example model defined for Qwen/Qwen-7B-Chat)。
实现拆解
- 在
tests/models/language/generation/test_common.py 中,从 AITER_MODEL_LIST 删除 "Qwen/Qwen-7B-Chat" 条目(第 28 行)。同时在 test_models 函数的参数化列表中删除对应的 pytest.param("Qwen/Qwen-7B-Chat", ...) 条目(原第 86-87 行)。
- 在
tests/distributed/test_pipeline_parallel.py 中,从 TEXT_ONLY_MODELS 字典删除 "Qwen/Qwen-7B-Chat": PPTestSettings.fast() 条目(原第 155 行)。
- 两个修改均为纯删除操作,无新增代码,确保测试套件不会因缺失的模型定义而失败。
关键文件:
tests/models/language/generation/test_common.py(模块 测试;类别 test;类型 test-coverage;符号 AITER_MODEL_LIST, test_models): 主要变更文件,删除了 AITER_MODEL_LIST 和 test_models 参数化列表中的 Qwen/Qwen-7B-Chat 条目,修复 CI 错误。
tests/distributed/test_pipeline_parallel.py(模块 测试;类别 test;类型 test-coverage;符号 TEXT_ONLY_MODELS): 删除了流水线并行测试字典中的 Qwen/Qwen-7B-Chat 条目,确保测试一致性。
关键符号:未识别
关键源码片段
tests/models/language/generation/test_common.py
主要变更文件,删除了 AITER_MODEL_LIST 和 test_models 参数化列表中的 Qwen/Qwen-7B-Chat 条目,修复 CI 错误。
# 从 AITER_MODEL_LIST 中移除 Qwen/Qwen-7B-Chat
AITER_MODEL_LIST = [
"meta-llama/Llama-3.2-1B-Instruct",
"openbmb/MiniCPM3-4B",
# "Qwen/Qwen-7B-Chat", # 已删除
"Qwen/Qwen2.5-0.5B-Instruct",
"TitanML/tiny-mixtral",
"Qwen/Qwen3-8B",
]
# 从 test_models 参数化列表中移除对应的 pytest.param
@pytest.mark.parametrize(
"model",
[
# ... 其他模型 ...
pytest.param(
"microsoft/phi-2", # phi
marks=[pytest.mark.core_model, pytest.mark.slow_test],
),
# pytest.param("Qwen/Qwen-7B-Chat", ...), # 已删除
pytest.param(
"Qwen/Qwen2.5-0.5B-Instruct", # qwen2
marks=[pytest.mark.core_model, pytest.mark.cpu_model, pytest.mark.slow_test],
),
# ...
],
)
tests/distributed/test_pipeline_parallel.py
删除了流水线并行测试字典中的 Qwen/Qwen-7B-Chat 条目,确保测试一致性。
TEXT_ONLY_MODELS = {
# ... 其他模型 ...
"microsoft/Phi-3.5-MoE-instruct": PPTestSettings.detailed(
multi_node_only=True, load_format="dummy"
),
# "Qwen/Qwen-7B-Chat": PPTestSettings.fast(), # 已删除
"Qwen/Qwen2.5-0.5B-Instruct": PPTestSettings.fast(),
"Qwen/Qwen1.5-MoE-A2.7B-Chat": PPTestSettings.fast(),
# ...
}
评论区精华
本 PR 无任何 review 评论,两位审核人均直接批准(APPROVED),表明变更简单且无争议。
风险与影响
- 风险:风险极低。变更仅涉及删除测试中已不再支持的模型引用,不会影响任何功能逻辑。但需注意如果未来重新引入 Qwen/Qwen-7B-Chat 支持,需同步恢复这些测试条目,否则可能导致测试覆盖缺失。
- 影响:直接影响:修复 CI 构建失败,允许流水线正常通过。间接影响:Qwen/Qwen-7B-Chat 模型的测试覆盖被移除,但该模型本身已在 #45131 中移除,因此测试删除是合理的清理操作。对用户无功能影响。
- 风险标记:测试覆盖缺失(因模型移除)
关联脉络
- PR #45131 [Removal] Drop Qwen/Qwen-7B-Chat model support: 本 PR 的动机源于 #45131 移除了 Qwen/Qwen-7B-Chat 模型支持,因此需要清理相关测试引用。
参与讨论