[None][infra] Waive 7 failed cases for main in post-merge 2938 - #18420
Conversation
Bug(s): 6693989, 6693991 Requested by: @mzweilz Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
|
/bot skip --comment "waive only" |
|
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; 10 remain after this review. WalkthroughThe integration waiver list adds skip entries for one LLM API streaming metadata test and six VERL tests. The entries reference NVBug reports covering multimodal generation, unimodal generation, and wake/sleep cycling. ChangesIntegration test waivers
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This change only waives seven known test cases in the integration test list; no actionable merge-blocking risk remains beyond normal review and checks. Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Description checkExplanation The description explains the automated waiver purpose, identifies the target branch and associated bugs, and lists all seven skipped test cases. It does not include the template headings for Description, Test Coverage, or the PR Checklist. 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 #70234 [ skip ] triggered by Bot. Commit: |
|
PR_Github #70234 [ skip ] completed with state |
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by @mzweilz)
Target branch:
mainBug(s): 6693989, 6693991
Waive entries added
This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.
Dev Engineer Review
SKIPwaivers intests/integration/test_lists/waives.txt.unittest/llmapi/test_llm.pycase and sixverl/test_verl_cases.pycases.QA Engineer Review
tests/integration/test_lists/waives.txt.