执行摘要

• 一句话：修复 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 的过滤修复可以防止此类回归再次发生。