[None][infra] Waive 1 failed cases for main in pre-merge 58439 - #18685
Conversation
Bug(s): 6720250 Requested by: @cascade812 Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
|
/bot skip --comment "waive PR" |
|
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; 11 remain after this review. WalkthroughThe integration test waiver list now skips ChangesTest waiver
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This change skips the specified visual-generation serving endpoints test under its tracked NVBug. The waiver format and target path are valid, with no remaining merge-readiness risk. Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Title checkExplanation The title clearly identifies the infrastructure change: adding a waiver for one failed pre-merge case on main. It is specific and related to the changeset, although the wording could be grammatically improved. Full details: Description checkExplanation The description explains the waiver, identifies the affected test, records NVBug 6720250, and states that the PR was auto-generated. It does not include the template's explicit Test Coverage section or completed checklist, but the relevant information is complete for this waiver-only change. Full details: Docstring CoverageExplanation 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)
Comment |
|
PR_Github #71351 [ skip ] triggered by Bot. Commit: |
|
PR_Github #71351 [ skip ] completed with state |
Picks up the main-side waive of unittest/_torch/visual_gen/test_trtllm_serve_endpoints.py (NVIDIA#18685) that failed the CPU lanes in the previous CI round. Made-with: Claude Code (Fable 5.1) Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Signed-off-by: longcheng-nv <243710427+longcheng-nv@users.noreply.github.com>
Picks up the main-side waive of unittest/_torch/visual_gen/test_trtllm_serve_endpoints.py (NVIDIA#18685) that failed the CPU lanes in the previous CI round. Made-with: Claude Code (Fable 5.1) Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Signed-off-by: longcheng-nv <243710427+longcheng-nv@users.noreply.github.com>
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by @cascade812)
Target branch:
mainBug(s): 6720250
Waive entries added
This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.
Dev Engineer Review
tests/integration/test_lists/waives.txt.unittest/_torch/visual_gen/test_trtllm_serve_endpoints.py.6720250.QA Engineer Review
tests/integration/test_lists/waives.txt.unittest/_torch/visual_gen/test_trtllm_serve_endpoints.py.