Prhub

#44605 [CI/Build] Disable CPU-Compatibility Tests

原始 PR 作者 bigPYJ1151 合并时间 2026-06-05 13:14 文件变更 1 提交数 1 评论 0 代码增减 +13 / -12

执行摘要

禁用因网络策略不可用的 CPU 兼容性测试

PR body 指出:"The tests dependency can't be downloaded from CI hosts because of a new firewall policy." 即CI主机因新的防火墙策略,无法下载Intel SDE依赖,导致CPU兼容性测试始终失败,因此需要暂时禁用。

该PR无需精读,属于运维层面的临时修复。值得关注的点是:团队采用了注释而非删除的方式保留配置,体现了良好的可恢复性设计。

讨论亮点

该PR没有review评论,仅有一条来自jikunshang的批准(APPROVED)和一条claude[bot]的自动评论(因来自fork而跳过review)。无实质性讨论。

实现拆解

  1. 修改CI配置文件:修改 .buildkite/hardware_tests/cpu.yaml,将 CPU-Compatibility Tests 整个步骤块(从 - label: CPU-Compatibility Tests 到对应的 bash 命令结束)注释掉。
  2. 添加注释说明:在注释块上方新增一行注释:# Note: SDE can't be downloaded from CI host because of AWS WAF,说明禁用原因。
  3. 保留其他测试步骤:除 CPU-Compatibility Tests 外,CPU-Kernel TestsCPU-Language Generation and Pooling Model TestsCPU-ModelRunnerV2 Tests 等步骤保持不变。
  4. 无其他文件变更:仅修改该YAML文件,未涉及源代码、测试脚本或脚本逻辑。
文件 模块 状态 重要度
.buildkite/hardware_tests/cpu.yaml CI 配置 modified 4.12

关键源码片段

.buildkite/hardware_tests/cpu.yaml test-coverage

唯一修改的文件,通过注释禁用了 `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"

评论区精华

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

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

风险与影响

低风险。仅注释了CI中的一个测试步骤,不会影响任何源代码或运行时行为。唯一风险是:如果未来防火墙策略解除,需要有人记得恢复该测试;当前注释保留了完整配置,方便日后取消注释。

影响范围很小:

  • 影响对象:仅影响CPU硬件CI流水线,移除了 CPU-Compatibility Tests 步骤。
  • 影响程度:低。该测试原本用于验证CPU兼容性(可能涉及不同指令集模拟),禁用后CI中相关兼容性验证缺失,但不会影响其他测试或产品代码。
  • 临时性:变更以注释方式实施,便于日后恢复。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论