[None][infra] Waive 3 failed cases for main in post-merge 2658#13141
Conversation
Bug(s): 6088149 Requested by: @mzweilz Signed-off-by: ZhanruiSunCh <184402041+ZhanruiSunCh@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR adds three new test skip entries to the waives list for performance sanity tests. Two entries skip disaggregated upload generation-only tests for gb200_deepseek-r1-fp4_1k1k with different configurations, and one entry skips an aggregated upload test for k25_thinking_fp4. All are marked with the same bug reference. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
/bot skip --comment "Waive PR created via CI report, skip testing." |
|
PR_Github #43926 [ skip ] triggered by Bot. Commit: |
|
PR_Github #43926 [ skip ] completed with state |
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by @mzweilz)
Target branch:
mainBug(s): 6088149
Waive entries added
This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.
Summary by CodeRabbit