Repository navigation
[None][infra] Waive 1 failed cases for main in pre-merge 65959 - #20960
Conversation
Bug(s): 6988813 Requested by: @chzblych Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
|
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. |
|
/bot skip --comment "Test waive" |
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Add test skip entry tests/integration/test_lists/waives.txt |
Adds a skip-list entry for the synchronous remote MPI session test, tracked by issue 6988813. |
Priority:
Estimated code review effort: 1 (Trivial) | ~2 minutes
Change: Other
Suggested reviewers: brnguyen2
Merge Risk: 
1cf70
-
The waiver is scoped to the reported failing test and references bug 6988813.
Pre-merge checks: 
-
Check name Status Explanation Title check The title clearly identifies an infrastructure waiver for one failed case on the main branch and references pre-merge run 65959. Description check The description explains the waiver, identifies the skipped test, cites NVBugs 6988813, and provides the target branch and requester. It does not include the template headings or checklist, but the re… Docstring Coverage No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request. Full details: Description check
-
Explanation
The description explains the waiver, identifies the skipped test, cites NVBugs 6988813, and provides the target branch and requester. It does not include the template headings or checklist, but the relevant information is complete for this auto-generated waiver.
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
|
PR_Github #80451 [ skip ] triggered by Bot. Commit: |
|
PR_Github #80451 [ skip ] completed with state |
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by @chzblych)
Target branch:
mainBug(s): 6988813
Waive entries added
This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.
Dev Engineer Review
The waiver list skips
unittest/llmapi/test_mpi_session.py::test_remote_mpi_session[submit_sync]and links it to bug 6988813. The change does not alter test code or runtime behavior. The test will not run while this waiver applies.QA Engineer Review
The change modifies
tests/integration/test_lists/waives.txt. It adds a waiver fortest_remote_mpi_session[submit_sync]; it does not add or modify test functions. The test list already contains waivers for the[submit]and[task_kwargs]cases. Test results are not provided. Coverage verdict: needs follow-up.Per-File QA Perspective
tests/integration/test_lists/waives.txt: Adds a skip for thesubmit_synccase. QA should track bug 6988813 and confirm the case can run after the issue is resolved.