执行摘要
- 一句话:禁用因网络策略不可用的CPU兼容性测试
- 推荐动作:该PR无需精读,属于运维层面的临时修复。值得关注的点是:团队采用了注释而非删除的方式保留配置,体现了良好的可恢复性设计。
功能与动机
PR body 指出:"The tests dependency can't be downloaded from CI hosts because of a new firewall policy." 即CI主机因新的防火墙策略,无法下载Intel SDE依赖,导致CPU兼容性测试始终失败,因此需要暂时禁用。
实现拆解
- 修改CI配置文件:修改
.buildkite/hardware_tests/cpu.yaml,将 CPU-Compatibility Tests 整个步骤块(从 - label: CPU-Compatibility Tests 到对应的 bash 命令结束)注释掉。
- 添加注释说明:在注释块上方新增一行注释:
# Note: SDE can't be downloaded from CI host because of AWS WAF,说明禁用原因。
- 保留其他测试步骤:除
CPU-Compatibility Tests 外,CPU-Kernel Tests、CPU-Language Generation and Pooling Model Tests、CPU-ModelRunnerV2 Tests 等步骤保持不变。
- 无其他文件变更:仅修改该YAML文件,未涉及源代码、测试脚本或脚本逻辑。
关键文件:
.buildkite/hardware_tests/cpu.yaml(模块 CI配置;类别 test;类型 test-coverage): 唯一修改的文件,通过注释禁用了 CPU-Compatibility Tests 步骤并添加了原因说明。
关键符号:未识别
关键源码片段
.buildkite/hardware_tests/cpu.yaml
唯一修改的文件,通过注释禁用了 CPU-Compatibility Tests 步骤并添加了原因说明。
# Note: SDE can't be downloaded from CI host because of AWS WAF
# 由于 AWS WAF 防火墙策略,Intel SDE 依赖无法从 CI 主机下载,
# 因此暂时禁用 CPU-Compatibility Tests。后续待防火墙策略调整后,
# 取消注释即可恢复。
# - label: CPU-Compatibility Tests
# depends_on: []
# device: intel_cpu
# no_plugin: true
# source_file_dependencies:
# - cmake/cpu_extension.cmake
# - setup.py
# - vllm/platforms/cpu.py
# commands:
# - |
# bash .buildkite/scripts/hardware_ci/run-cpu-test.sh 20m "
# bash .buildkite/scripts/hardware_ci/run-cpu-compatibility-test.sh"
评论区精华
该PR没有review评论,仅有一条来自jikunshang的批准(APPROVED)和一条claude[bot]的自动评论(因来自fork而跳过review)。无实质性讨论。
风险与影响
- 风险:低风险。仅注释了CI中的一个测试步骤,不会影响任何源代码或运行时行为。唯一风险是:如果未来防火墙策略解除,需要有人记得恢复该测试;当前注释保留了完整配置,方便日后取消注释。
- 影响:影响范围很小:
- 影响对象:仅影响CPU硬件CI流水线,移除了
CPU-Compatibility Tests 步骤。
- 影响程度:低。该测试原本用于验证CPU兼容性(可能涉及不同指令集模拟),禁用后CI中相关兼容性验证缺失,但不会影响其他测试或产品代码。
- 临时性:变更以注释方式实施,便于日后恢复。
- 风险标记:暂无
关联脉络
参与讨论