Prhub

#21410 [Bugfix] Disable ci for .md files

原始 PR 作者 OrangeRedeng 合并时间 2026-03-26 01:19 文件变更 6 提交数 14 评论 5 代码增减 +16 / -0

执行摘要

修复 CI 工作流中 .md 文件触发测试的 bug,避免不必要的 CI 运行。

修复Issue 21409,具体问题未提供细节,但基于PR描述,是防止.md文件更改触发CI测试,以减少资源浪费。

这是一个简单的CI配置修复,建议快速审核以确保路径过滤规则正确,无需深入代码学习;对CI维护者和基础设施团队有价值,可关注类似PR #21334以理解CI优化趋势。

讨论亮点

在Issue评论中,Fridge003指出'This commit will unexpectedly trigger every test',担忧修改后路径过滤错误触发测试。hnyls2002建议参考PR #21334修改规则,随后ping1jing2确认ok,问题得到解决。讨论焦点在于确保CI触发逻辑的正确性,并借鉴已有最佳实践。

实现拆解

修改了6个.github/workflows下的CI配置文件:pr-test-amd-rocm720.yml、pr-test-amd.yml、pr-test-npu.yml、pr-test-xeon.yml、pr-test-xpu.yml、pr-test.yml。在每个文件的路径过滤部分添加了'!*/.md'条目,以排除所有.md文件变更触发CI测试,适用于sgl_kernel、jit_kernel、multimodal_gen等模块的CI触发规则。

文件 模块 状态 重要度
.github/workflows/pr-test-amd-rocm720.yml CI Infrastructure modified 5.0
.github/workflows/pr-test-amd.yml CI Infrastructure modified 5.0
.github/workflows/pr-test-npu.yml CI Infrastructure modified 5.0
.github/workflows/pr-test-xeon.yml CI Infrastructure modified 5.0
.github/workflows/pr-test-xpu.yml CI Infrastructure modified 5.0
.github/workflows/pr-test.yml CI Infrastructure modified 5.0

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

评论区精华

CI 触发逻辑修改是否正确 设计

Fridge003 评论说 'This commit will unexpectedly trigger every test',表达了对修改后路径过滤可能错误触发所有测试的担忧。hnyls2002 建议参考 PR #21334 修改规则,以借鉴类似 CI 优化。

结论:ping1jing2 回复 ok,表示问题解决,修改被接受。 · 已解决

风险与影响

风险较低。主要风险是路径过滤语法'!*/.md'可能在某些工作流中不生效或导致其他必要文件被错误排除,从而影响CI触发准确性。已有讨论中提到了潜在触发问题,但通过参考#21334的类似规则解决,减少了风险。

对最终用户无直接影响。对团队和系统影响:优化CI触发逻辑,减少因.md文件更改引发的不必要测试运行,提高CI效率和资源利用率;影响范围限于CI基础设施模块,程度为中等改进。

路径过滤错误 CI 配置变更

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论