Repository navigation
[None][infra] Waive 1 failed cases for main in pre-merge 65251 - #20678
Conversation
Bug(s): 6960771 Requested by: @jiaganc Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
|
/bot skip --comment "waive 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
WalkthroughThe integration test skip list now includes ChangesIntegration test waiver
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~2 minutes Change: Other Suggested reviewers: Merge Risk: ⚪ Minimal · up to
|
| Check name | Status | Explanation |
|---|---|---|
| Title check |  |
The title follows the required [ticket][type] format and clearly identifies an infrastructure PR that waives one failed case on main. |
| Description check | |
The description identifies the requester, target branch, NVBug, and exact test waiver. It does not use the template headings or include a separate test coverage and checklist section, but it provides … |
| 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 identifies the requester, target branch, NVBug, and exact test waiver. It does not use the template headings or include a separate test coverage and checklist section, but it provides the key information for this auto-generated waiver PR.
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 #79241 [ skip ] triggered by Bot. Commit: |
|
PR_Github #79241 [ skip ] completed with state |
3 participants
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by @jiaganc)
Target branch:
mainBug(s): 6960771
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
SKIPwaiver fortest_llm_kv_events_apiand links it to NVBug6960771. It changes no source code or public API. The waiver suppresses coverage for this API test until the bug is resolved.QA Engineer Review
tests/integration/test_lists/waives.txtadds the waiver forunittest/llmapi/test_llm_kv_cache_events.py::test_llm_kv_events_api. No test implementation changed. Coverage verdict: needs follow-up because the test is skipped.Per-File QA Perspective
tests/integration/test_lists/waives.txt: Adds theSKIPentry fortest_llm_kv_events_api. QA should remove the waiver and confirm the test passes after NVBug6960771is fixed.