Repository navigation
[None][infra] Waive 2 failed cases for main in pre-merge 66900 - #21247
Conversation
Bug(s): 7011346 Requested by: @HuiGao-NV 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 "waive 2 flaky timeout issue with high failure rate" |
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Add skip-list entries tests/integration/test_lists/waives.txt |
The skip list adds entries for cancel_late_aux and cancel_late_kv, each linked to nvbugs/7011346. |
Priority:
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Other
Possibly related PRs
- NVIDIA/TensorRT-LLM#21219: Changes worker startup and pytest timeout budgets in the test containing the waived cases.
- NVIDIA/TensorRT-LLM#20439: Removed waivers for the same two cases.
Suggested reviewers: xinhe-nv, chienchunhung
Merge Risk: 
763fa
-
The targeted flaky timeout cases are waived under the referenced bug, with no remaining merge-blocking risk.
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 for pre-merge 66900. Description check The description explains that the PR is auto-generated, identifies the target branch and bug, and lists both waive entries. It does not include the template headings for Description, Test Coverage, or… 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 explains that the PR is auto-generated, identifies the target branch and bug, and lists both waive entries. It does not include the template headings for Description, Test Coverage, or PR Checklist, but the required change context is clear for this waiver-only 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.
|
Add Carrot credits or activate Agent usage billing to use Autofix |
|
PR_Github #81366 [ skip ] triggered by Bot. Commit: |
|
PR_Github #81366 [ skip ] completed with state |
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by @HuiGao-NV)
Target branch:
mainBug(s): 7011346
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
SKIPwaivers intests/integration/test_lists/waives.txtfor thecancel_late_auxandcancel_late_kvtransfer lifecycle cases. Both entries reference bug 7011346. The change affects test selection only; no source or API changes are reported.Review finding counts are unavailable from the supplied evidence.
QA Engineer Review
The change modifies only
tests/integration/test_lists/waives.txt. It addsSKIPentries fortest_transfer_lifecycle_gpu[cancel_late_aux]andtest_transfer_lifecycle_gpu[cancel_late_kv]. No test code changed. These cases are therefore waived rather than run through this list. Coverage verdict: needs follow-up.Per-File QA Perspective
tests/integration/test_lists/waives.txt: Adds two bug-linked skip entries for GPU transfer lifecycle cases. Verify that the waivers are removed when the reported timeout issues are resolved.