Prhub
← 返回仓库详情

#21410 [Bugfix] Disable ci for .md files

sgl-project/sglang · 作者 OrangeRedeng · 合并时间 2026-03-26 01:19

分析状态 已生成
文件变更 6提交数 14 · 评论 5
代码增减 +16 / -0
bugfix ci

执行摘要

修复 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 链接，后续同步到相关引用后会出现在这里。

完整报告

Markdown PDF

执行摘要

本PR通过修改多个CI工作流配置文件，添加.md文件排除规则，修复了.md文件更改意外触发CI测试的bug，优化了CI触发逻辑，以减少不必要的测试运行并节省资源。

功能与动机

修复Issue 21409，具体动机是防止.md文件（如文档文件）的变更触发CI测试，以避免资源浪费。从PR body中引用："Fix of https://github.com/sgl-project/sglang/issues/21409"，但Issue详情未提供，推断为优化CI效率。

实现拆解

修改了6个CI配置文件，均位于.github/workflows/目录下，属于CI基础设施模块。关键改动是在每个文件的路径过滤部分添加!**/*.md条目，以排除所有.md文件。例如，在.github/workflows/pr-test.yml中：

- "!**/*.md"

类似修改应用于其他工作流文件，覆盖AMD、NPU、Xeon、XPU等硬件平台的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

评论区精华

主要讨论发生在Issue评论中：

  • Fridge003 指出："This commit will unexpectedly trigger every test"，担忧修改可能导致CI错误触发。
  • hnyls2002 建议："@Kangyan-Zhou @ping1jing2 @OrangeRedeng Please change the rules similar to https://github.com/sgl-project/sglang/pull/21334"，推荐参考PR #21334的类似规则以确保正确性。
  • ping1jing2 确认："ok"，表示问题解决，修改被接受。

讨论焦点在于CI触发逻辑的设计权衡，通过借鉴已有实践来降低风险。

风险与影响

风险分析：风险较低，但需注意路径过滤语法!**/*.md的正确性，确保不会错误排除其他必要文件或导致CI触发失效。已有讨论中提到潜在触发问题，但参考#21334后缓解。

影响分析：对用户无直接影响；对系统影响正面，优化CI触发逻辑，减少不必要的测试运行，提高资源利用率；影响范围限于CI基础设施，程度为中等改进。

关联脉络

与历史PR #21334相关，后者可能涉及类似的CI触发规则调整（如收窄触发范围），显示团队在持续优化CI配置以减少噪音。从近期历史PR分析看，如PR #21385（重构扩散JIT内核测试布局并收窄CI触发）和PR #21455（简化CI检查动作），本PR是CI优化趋势的一部分，旨在提高开发效率。

参与讨论