#7125 [ci] fix: tigger npu ci on the release branch

原始 PR 作者 tardis-key 合并时间 2026-07-23 12:03 文件变更 2 提交数 1 评论 1 代码增减 +2 / -0

执行摘要

CI配置缺失导致向release分支提交的PR无法自动触发NPU测试。PR body明确说明需要更严格的release分支检查。commit message指出#7123等PR因修改NPU相关代码（如 verl/models/mcore/util.py）但无法触发NPU CI。

可快速合并。推荐运维或CI负责人关注此改动是否覆盖所有release分支命名模式，必要时可增加更通用的分支匹配规则（如 release/**）。

讨论亮点

无review评论。wucong25直接批准，无讨论。

在 .github/workflows/e2e_ascend.yml 的 on.pull_request.branches 列表中添加 - v0.*，使得PR目标为 release 分支（以 v0. 开头）时也能触发端到端 Ascend CI。
在 .github/workflows/npu_unit_tests.yml 的 on.pull_request.branches 列表中添加相同的 - v0.*，使NPU单元测试CI同样覆盖 release 分支。
两个文件均只改动一行（+1/-0），结构一致，改动量极小。

文件	模块	状态	重要度
`.github/workflows/e2e_ascend.yml`	CI 配置	modified	2.64
`.github/workflows/npu_unit_tests.yml`	CI 配置	modified	2.64

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

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

低风险。仅在CI触发条件中增加分支匹配，不改变任何逻辑、测试或构建流程。但需确认仓库release分支确实以 v0.* 命名模式，若实际存在 release/* 或 v* 等不同命名，则此改动可能无法覆盖所有目标分支。

对用户无影响。对CI系统：从仅 main 分支触发扩展为 main 和 v0.* 分支触发，增加CI执行频次，可能少量增加CI资源消耗。对团队：确保release分支的NPU相关PR能自动通过CI检测，降低合并风险。

CI 配置变更分支匹配模式依赖仓库命名规范

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

在 .github/workflows/e2e_ascend.yml 的 on.pull_request.branches 列表中添加 - v0.*，使得PR目标为 release 分支（以 v0. 开头）时也能触发端到端 Ascend CI。
在 .github/workflows/npu_unit_tests.yml 的 on.pull_request.branches 列表中添加相同的 - v0.*，使NPU单元测试CI同样覆盖 release 分支。
两个文件均只改动一行（+1/-0），结构一致，改动量极小。

关键文件：

.github/workflows/e2e_ascend.yml（模块 CI配置；类别 infra；类型 infrastructure）: 在pull_request触发分支列表中添加v0.*，启用release分支的端到端Ascend CI。
.github/workflows/npu_unit_tests.yml（模块 CI配置；类别 infra；类型 infrastructure）: 与e2e_ascend.yml相同的改动，确保NPU单元测试同样覆盖release分支。

关键符号：未识别

无review评论。wucong25直接批准，无讨论。

风险：低风险。仅在CI触发条件中增加分支匹配，不改变任何逻辑、测试或构建流程。但需确认仓库release分支确实以 v0.* 命名模式，若实际存在 release/* 或 v* 等不同命名，则此改动可能无法覆盖所有目标分支。
影响：对用户无影响。对CI系统：从仅 main 分支触发扩展为 main 和 v0.* 分支触发，增加CI执行频次，可能少量增加CI资源消耗。对团队：确保release分支的NPU相关PR能自动通过CI检测，降低合并风险。
风险标记：CI配置变更, 分支匹配模式依赖仓库命名规范