[None][test] Waive 7 failed cases for main in QA CI - #19408
Conversation
Bug(s): 6672360, 6758594, 6776338, 6799208, 6801098, 6801102, 6801419 Requested by: qa@nvidia.com Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
|
/bot skip --comment "waive tests only" |
|
Automatically added "ci: full pre-merge approved" because this PR has satisfied the required GitHub review approvals. Unresolved review conversations and other required checks remain independent merge requirements. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 12 included reviews per hour; 9 remain after this review. WalkthroughThe integration waiver list adds seven GB200-specific entries for GPTOSS, MiniMax, Qwen, TinyLlama, and multimodal test cases. Each entry includes a corresponding NVIDIA bug reference. ChangesGB200 integration waivers
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~2 minutes Change: Other Possibly related PRs
Suggested reviewers: Merge Risk: ⚪ Minimal · up to The seven GB200 waiver entries are scoped and formatted as expected, so the change is ready to merge. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
PR_Github #74353 [ skip ] triggered by Bot. Commit: |
|
PR_Github #74353 [ skip ] completed with state |
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by qa@nvidia.com)
Target branch:
mainBug(s): 6672360, 6758594, 6776338, 6799208, 6801098, 6801102, 6801419
Waive entries added
This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.
Dev Engineer Review
The change adds seven GB200 entries to
tests/integration/test_lists/waives.txt. It does not change source code or runtime behavior. The waivers reduce CI coverage for the affected test cases until bugs 6672360, 6758594, 6776338, 6799208, 6801098, 6801102, and 6801419 are resolved.QA Engineer Review
The change modifies only the waiver list and adds seven GB200 waivers covering disaggregated, accuracy, LLM API, multimodal, and MoE LoRA cases. No test implementations or test functions changed. Coverage verdict: needs follow-up, because the change suppresses failed CI cases and no fix or waiver expiry is included.
Per-File QA Perspective
tests/integration/test_lists/waives.txt: Adds seven GB200 CI waivers linked to the seven reported bugs. QA should track the failures and remove the waivers after the fixes are verified; no relatedtest-db/orqa/list changes are shown.