Prhub

#25135 ci: merge sgl-kernel-build-wheels x86+arm into reusable workflow

原始 PR 作者 hnyls2002 合并时间 2026-05-14 04:52 文件变更 2 提交数 2 评论 2 代码增减 +137 / -125

执行摘要

合并 sgl-kernel 构建工作流至可复用模板

减少 CI 工作流中 x86 和 ARM wheel 构建的重复代码,统一维护入口,降低后续修改的出错概率。PR body 描述为“Mechanical refactor + 6-layer equivalence audit”,并指出唯一的两处声明性差异。

该 PR 是 CI 重构的优秀范例,尤其推荐关注其 6 层等价审计方法,可作为后续类似重构的参考。对于不直接参与 CI 维护的工程师,无需深究细节。

实现拆解

  1. 创建可复用工作流(_pr-test-sgl-kernel-build.yml):定义 on.workflow_call.inputs,包括 runs_on(运行器标签)、job_display_name(UI 显示名)、arch_suffix(制品后缀)等关键参数;镜像调用方 env 确保上下文一致。
  2. pr-test.yml 中替换原有内联构建作业:原本的 sgl-kernel-build-wheels(x86)和 sgl-kernel-build-wheels-arm 各自包含完整的环境、矩阵、步骤,现改为一行 uses 调用,通过 with 传入对应参数。
  3. 保留原有 needsif 条件不变:确保触发逻辑与原作业一致。
  4. 通过 6 层等价审计验证:包括调用方 needs/if 字节一致、with 键合法、可复用工作流 env 镜像、主体替换比对、以及逆向回环校验(将可复用主体回插后与 PR 基础版本 SHA256 一致)。
文件 模块 状态 重要度
.github/workflows/_pr-test-sgl-kernel-build.yml CI 工作流 added 6.13
.github/workflows/pr-test.yml CI 工作流 modified 5.26

分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。

评论区精华

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

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

风险与影响

由于作者执行了严格的 6 层等价审计(包括 SHA256 逆向回环验证),已确认只有两处声明性差异且行为等价,因此回归风险极低。唯一潜在风险是未来修改可复用工作流时可能影响两侧构建,但通过参数化设计已最小化耦合。

影响范围仅限 CI 配置(.github/workflows/),不影响运行时逻辑、模型推理或 API。对开发者无感知,但 CI 维护者将受益于降低的重复代码和维护成本。

仅 CI 配置变更 无功能变更 等价审计已验证

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论