fix(ci): restore parallel TileLib ST and watchdog labels - #1105
Merged
mouliangyu merged 2 commits intoAug 3, 2026
Conversation
mouliangyu
force-pushed
the
codex/fix-tilelib-st-watchdog
branch
from
August 3, 2026 10:52
5ebae15 to
31afbbb
Compare
mouliangyu
marked this pull request as ready for review
August 3, 2026 10:54
A5 板测成功
|
A3 板测失败
失败用例
|
A3 板测失败详情:PR #1105comm_p2p_binding_variants
comm_p2p
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
背景
PR #1028 已经将 TileLib ST 改为按 case 使用
pytest-xdist并行调度,但后续 CI workflow 修改重新使用了串行run_tilelib_st.py入口。当前 154 个 case 因此共享一个 simulator 会话串行执行;最近一次串行门禁中的Run TileLib ST CI用时约 59 分钟。同一次门禁超过 watchdog 的 90 分钟软预算后,watchdog 虽然正确识别了慢运行,却在给 PR 添加
ci-slowlabel 时因为GITHUB_TOKEN只有 pull-request read 权限而收到 403。该 job 的continue-on-error又让 Watchdog workflow 最终显示为 success,没有留下 label 或评论。修改
pytest-xdist --dist=worksteal调度独立 case。TILELIB_ST_WORKSPACE、PTODSL Python、PTOAS 和日志路径配置。issues: write和pull-requests: write,使GITHUB_TOKEN可以更新 PR 的ci-slowlabel 和 Conversation 评论。continue-on-error,使 watchdog 自身的权限或脚本错误在独立 workflow 中可见;该 workflow 不是ci-simrequired check,不会改变原门禁结果。验证
.github/workflows/ci_sim.yml和.github/workflows/watchdog.yml通过 PyYAML 解析。py_compile。pytest --collect-only -n 16 --dist=worksteal test/tilelib-st/test_tilelib_st.py通过。git diff --check通过。后续 CI 出现的 exit 143 已确认来自宿主机 OOM:PR #1105 的 TileLib simulator 与 PR #1104 的 PTOAS 编译等任务同时运行,runner 所在 tmux scope 使用
OOMPolicy=stop,action 子进程成为 OOM victim 后连带关闭了整个 runner。该宿主机部署问题不通过降低本 PR 的 TileLib 并行度处理。说明:
workflow_run会从默认分支加载 watchdog,因此新的 PR 写权限需要在本 PR 合入后,由下一次 PRci-sim完成事件做端到端验证。Follow-up to #1028 and #1031.