Repository navigation
[None][infra] Waive 2 failed cases for main in pre-merge 67069 - #21264
Conversation
Bug(s): 6902353 Requested by: @lori-ren Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
|
/bot skip --comment "waive only PR" |
|
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.
 |
| Layer / File(s) | Summary |
|---|---|
Add lifecycle test waivers tests/integration/test_lists/waives.txt |
Adds skip-list entries for source_fatal and timeout_late_aux, both referencing NVBUG 6902353. |
Priority:
Estimated code review effort: 1 (Trivial) | ~2 minutes
Change: Other
Suggested reviewers: xinhe-nv
Merge Risk: 
a7399
-
The waiver-only change is ready to merge.
Pre-merge checks: 
-
Check name Status Explanation Title check The title clearly identifies an infrastructure change that waives two failed cases on the main branch. Description check The description identifies the waiver purpose, target branch, bug ID, requester, and exact test cases. It omits the template headings and checklist, but it provides the critical information for this a… Docstring Coverage No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat… 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 identifies the waiver purpose, target branch, bug ID, requester, and exact test cases. It omits the template headings and checklist, but it provides the critical information for this auto-generated waiver PR.
Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. 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 #81446 [ skip ] triggered by Bot. Commit: |
|
PR_Github #81446 [ skip ] completed with state |
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by @lori-ren)
Target branch:
mainBug(s): 6902353
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 two skip entries in
tests/integration/test_lists/waives.txtforsource_fatalandtimeout_late_aux, both linked to NVBUG 6902353. The scope is limited to the test waiver list. These cases will not run while the waivers apply, so their behavior needs follow-up after the bug is resolved. No test results were supplied.QA Engineer Review
The waiver list adds entries for
test_transfer_lifecycle_gpu[source_fatal]andtest_transfer_lifecycle_gpu[timeout_late_aux]. Both cases are skipped due to NVBUG 6902353. No matching test entries were found in the suppliedtest-db/orqa/list search. Coverage verdict: needs follow-up.Per-File QA Perspective
tests/integration/test_lists/waives.txt: Adds skips for two GPU transfer lifecycle cases. Verify the waivers are removed when NVBUG 6902353 is resolved.