[None][infra] Waive 9 failed cases for main in post-merge#14515
Conversation
881ea5c to
c5733cf
Compare
|
/bot run --stage-list "" |
📝 WalkthroughWalkthroughUpdated ChangesTest Waivers for Known Issues
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@tests/integration/test_lists/waives.txt`:
- Around line 346-347: The two waiver lines include an incorrect class segment
"::stress_test::" in the pytest node IDs; update both entries so they use the
exact function-level node IDs (remove the "::stress_test::" segment) e.g. change
"stress_test/stress_test.py::stress_test::test_run_stress_test[...]" to
"stress_test/stress_test.py::test_run_stress_test[...]" for both occurrences so
validate-test-lists will match the collected tests.
🪄 Autofix (Beta)
❌ Autofix failed (check again to retry)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: f7a9c658-5fad-4273-a89b-ee4605b8e10d
📒 Files selected for processing (1)
tests/integration/test_lists/waives.txt
|
PR_Github #50153 [ run ] triggered by Bot. Commit: |
|
PR_Github #50153 [ run ] completed with state
|
|
Note Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it. An unexpected error occurred while generating fixes: Not Found - https://docs.github.com/rest/git/refs#get-a-reference |
94c9000 to
6255b97
Compare
|
/bot run --stage-list "" |
|
PR_Github #50164 [ run ] triggered by Bot. Commit: |
|
PR_Github #50164 [ run ] completed with state |
6255b97 to
f5458e7
Compare
|
/bot reuse-pipeline |
|
PR_Github #50183 [ reuse-pipeline ] triggered by Bot. Commit: |
|
PR_Github #50183 [ reuse-pipeline ] completed with state |
f5458e7 to
2153196
Compare
|
/bot reuse-pipeline |
|
PR_Github #50196 [ reuse-pipeline ] triggered by Bot. Commit: |
|
PR_Github #50196 [ reuse-pipeline ] completed with state |
Bug(s): 6185146, 6198785, 6207678, 6211185, 6211189, 6211693, 6215689, 6215736 Requested by: [email protected] Signed-off-by: xinhe-nv <[email protected]>
Signed-off-by: Jie Li <[email protected]>
2153196 to
34f7a91
Compare
|
/bot reuse-pipeline |
|
PR_Github #50209 [ reuse-pipeline ] triggered by Bot. Commit: |
|
PR_Github #50209 [ reuse-pipeline ] completed with state |
Signed-off-by: xinhe-nv <[email protected]>
Signed-off-by: xinhe-nv <[email protected]>
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by [email protected])
Target branch:
mainBug(s): 6185146, 6198785, 6207678, 6211185, 6211189, 6211693, 6215689, 6215736
Waive entries added
Already waived (skipped)
accuracy/test_llm_api_pytorch_multimodal.py::TestExaone4_5_33B::test_auto_dtype[full_budget]accuracy/test_llm_api_pytorch_multimodal.py::TestGemma3_27BInstruct::test_fp8_prequantizedaccuracy/test_llm_api_pytorch_multimodal.py::TestExaone4_5_33B::test_auto_dtype[forced_chunked_prefill]This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.
Summary by CodeRabbit