[None][infra] Waive 1 failed cases for main in pre-merge 60861 - #19343
Conversation
Bug(s): 6793964 Requested by: @xxi-nv Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
|
/bot skip --comment "waive the flaky test to unblock CI" |
|
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; 8 remain after this review. WalkthroughThe integration waiver list adds a skipped DGX H100 test case for the unbalanced two-device ChangesTest waiver
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~2 minutes Change: Other Suggested reviewers: Merge Risk: ⚪ Minimal · up to This PR only suppresses the specified known failing test on DGX H100 and records its tracking bug, so it is mergeable. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
PR_Github #74073 [ skip ] triggered by Bot. Commit: |
|
PR_Github #74073 [ skip ] completed with state |
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by @xxi-nv)
Target branch:
mainBug(s): 6793964
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 one waiver for
full:DGX_H100/unittest/_torch/multi_gpu/test_linear.py::test_row_linear[2-unbalanced]under NVBug 6793964. It has limited scope and does not change source code, APIs, or runtime behavior. Review findings: unavailable from the supplied evidence.QA Engineer Review
tests/integration/test_lists/waives.txtadds one skipped test entry. The waiver covers the unbalanced two-waytest_row_linearcase onDGX_H100; no test implementation changes are included. Coverage verdict: needs follow-up because the waiver suppresses a pre-merge case and the supplied evidence does not establish whether the underlying bug is resolved.Per-File QA Perspective
tests/integration/test_lists/waives.txt: The file adds a waiver for the identified integration test and NVBug 6793964. QA should verify that the waiver is correctly scoped and that the listed test remains tracked for re-enablement after the bug is fixed.