[CI] Remove flashinfer cache cleanup to reduce unit test runtime#7476
[CI] Remove flashinfer cache cleanup to reduce unit test runtime#7476EmmonsCurse wants to merge 1 commit intoPaddlePaddle:developfrom
Conversation
|
Thanks for your contribution! |
|
/skip-ci ci_iluvatar |
PaddlePaddle-bot
left a comment
There was a problem hiding this comment.
🤖 AI Code Review |
2026-04-18 20:59 CST
📋 Review 摘要
PR 概述:移除分布式测试中不必要的 flashinfer cache 清理步骤,以减少 CI 运行时间
变更范围:tests/layers/test_trtllm_allreduce_rms_fusion.py
影响面 Tag:CI
问题
未发现阻塞性问题。
总体评价
变更简洁安全。删除的 cache 清理逻辑(rm -rf /root/.cache/flashinfer)不影响测试正确性,flashinfer 库会自行管理其编译缓存。删除后 os、subprocess 等 import 仍被其他代码使用,无残留 dead import。LGTM。
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #7476 +/- ##
==========================================
Coverage ? 74.19%
==========================================
Files ? 398
Lines ? 54987
Branches ? 8616
==========================================
Hits ? 40800
Misses ? 11453
Partials ? 2734
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Motivation
The
flashinfer cachecleanup step was introducing unnecessary overhead during unit test execution.Since the cache removal is not required for test correctness, it leads to increased test runtime without providing additional value.
Modifications
/root/.cache/flashinfer) in the distributed test.Usage or Command
N/A
Accuracy Tests
N/A
Checklist
[FDConfig],[APIServer],[Engine],[Scheduler],[PD Disaggregation],[Executor],[Graph Optimization],[Speculative Decoding],[RL],[Models],[Quantization],[Loader],[OP],[KVCache],[DataProcessor],[BugFix],[Docs],[CI],[Optimization],[Feature],[Benchmark],[Others],[XPU],[HPU],[GCU],[DCU],[Iluvatar],[Metax]]pre-commitbefore commit.releasebranch, make sure the PR has been submitted to thedevelopbranch, then cherry-pick it to thereleasebranch with the[Cherry-Pick]PR tag.