# PR #27222 完整报告

- 仓库：`sgl-project/sglang`
- 标题：[CI] Fix multimodal-gen path filter for shared trace code
- 合并时间：2026-06-04 14:20
- 原文链接：http://prhub.com.cn/sgl-project/sglang/pull/27222

---

# 执行摘要

- 一句话：修复 CI 路径过滤器遗漏 observability 目录
- 推荐动作：该 PR 改动极小，但修复了关键的 CI 过滤漏洞。建议：
 - 审核 CI 配置的路径过滤逻辑，确认是否有类似遗漏（如其他共享模块目录是否被正确包含）。
 - 建议精简 CI 工作流中重复的过滤条件，避免后续维护负担。

# 功能与动机

PR #23755 将 observability 代码移至 srt/observability，但未更新 CI 路径过滤器，导致修改 observability 文件（如 trace.py）时 `check-changes` 设置 `multimodal_gen=false`，跳过 multimodal-gen 工作流（含 multimodal-gen-unit-test），使 `test_disagg_trace.py` 等回归未被捕获（需后续修复 #27173）。该 PR 是为了修复此过滤漏洞。

# 实现拆解

该 PR 实现简单，仅涉及 CI 工作流文件的路径过滤条件修改：

1. **修改三个 CI 工作流文件**：在 `_pr-test-check-changes.yml`, `pr-test-amd-rocm720.yml`, `pr-test-amd.yml` 中的 `multimodal_gen` 过滤条件列表里，分别增加一行 `- "python/sglang/srt/observability/**"`。

2. **不改动任何源代码**：所有修改仅限 `.github/workflows/` 下的 YAML 配置，无运行时影响。

3. **测试覆盖**：无单独测试，但通过后续 CI 运行验证效果。

关键文件：
- `.github/workflows/_pr-test-check-changes.yml`（模块 CI 配置；类别 infra；类型 infrastructure）: 核心变更文件：定义了 `check-changes` job 的路径过滤逻辑，新增 observability 路径以确保修改 trace 共享代码时触发 multimodal-gen 工作流。
- `.github/workflows/pr-test-amd-rocm720.yml`（模块 CI 配置；类别 infra；类型 infrastructure）: AMD ROCm720 测试工作流，同步添加 observability 路径过滤，保证 AMD CI 的正确性。
- `.github/workflows/pr-test-amd.yml`（模块 CI 配置；类别 infra；类型 infrastructure）: AMD 常规测试工作流，同步添加 observability 路径过滤，保证 AMD CI 的正确性。

关键符号：未识别


# 评论区精华

该 PR 无 review 评论或讨论线程。变更由作者自行合并，说明修复的紧迫性以及社区的共识接受。

- 暂无高价值评论线程

# 风险与影响

- 风险：风险极低。
- **回归风险**：仅添加路径通配符，不会引发系统运行问题；但需确保该通配符不会过度触发 multimodal-gen 工作流（仅 observability 目录变化会触发，影响合理）。
- **性能风险**：无。
- **安全风险**：无。
- 影响：**影响范围**：
 - **用户**：无直接影响。
 - **CI 系统**：修改 observability 目录下的文件（如 trace.py, mooncake_trace.py）时会正确触发 multimodal-gen 工作流；但不改变其他触发条件。
 - **团队**：避免了因路径过滤缺失导致 multimodal-gen 测试被跳过的问题，有利于维持 CI 质量。影响程度低。
 - 风险标记：暂无

# 关联脉络

- PR #23755 trace_modules gate disabling default trace contexts: PR #23755 引入了 `--trace-modules` 过滤并重构了 observability 代码路径（移至 srt/observability），但未更新 CI 路径过滤，导致本 PR 需要修复该遗漏。
- PR #27173 Fix trace_modules gate disabling default trace contexts: PR #27173 是修复因路径过滤缺失导致的 test_disagg_trace.py 回归的直接后续修复。本 PR 的过滤修复可以防止此类回归再次发生。