[None][test] Waive 9 failed cases for main in QA CI#14792
Conversation
Bug(s): 6059036, 6181383, 6185150, 6211441, 6223556, 6241842, 6241845, 6245279, 6245765 Requested by: qa@nvidia.com Jenkins build: https://prod.blsm.nvidia.com/swqa-tensorrt-qa-test/job/LLM_FUNCTION_CLUSTER_TEST/1485/ Signed-off-by: tensorrt-cicd <90828364+tensorrt-cicd@users.noreply.github.com>
Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
|
/bot run --skip-test |
📝 WalkthroughWalkthroughThis PR adds 8 new test skip entries to ChangesTest Waivers
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 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: 2
🤖 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`:
- Line 34: Update the nvbug URL for the skipped case named fp4_indexer_dsl_mtp3
in tests/integration/test_lists/waives.txt: replace the current
https://nvbugs/6241842 reference with the correct https://nvbugs/6241845 so the
waiver for
accuracy/test_llm_api_pytorch.py::TestDeepSeekV32::test_nvfp4_multi_gpus[fp4_indexer_dsl_mtp3]
points to the PR-intended bug ID.
- Around line 119-122: The waiver file tests/integration/test_lists/waives.txt
is missing the expected entry for the Nemotron test; add the line matching the
existing format for the skipped test:
accuracy/test_llm_api_pytorch.py::TestNemotronV3Super::test_nvfp4_8gpus[attention_dp_on-trtllm]
SKIP (https://nvbugs/6245765) to the file so the waive batch includes the full
9-case set referenced in the PR objective.
🪄 Autofix (Beta)
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: dced719c-ce81-4695-b0e2-20b334ae8133
📒 Files selected for processing (1)
tests/integration/test_lists/waives.txt
|
/bot run --skip-test |
|
PR_Github #51320 [ run ] triggered by Bot. Commit: |
Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
|
PR_Github #51320 [ run ] completed with state |
|
/bot reuse-pipeline |
|
/bot reuse-pipeline |
|
/bot reuse-pipeline |
1 similar comment
|
/bot reuse-pipeline |
|
/bot reuse-pipeline |
|
/bot reuse-pipeline |
6 similar comments
|
/bot reuse-pipeline |
|
/bot reuse-pipeline |
|
/bot reuse-pipeline |
|
/bot reuse-pipeline |
|
/bot reuse-pipeline |
|
/bot reuse-pipeline |
Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
|
/bot reuse-pipeline |
|
PR_Github #51557 [ reuse-pipeline ] triggered by Bot. Commit: |
|
PR_Github #51557 [ reuse-pipeline ] completed with state |
Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
|
/bot reuse-pipeline |
|
PR_Github #51712 [ reuse-pipeline ] triggered by Bot. Commit: |
|
PR_Github #51712 [ reuse-pipeline ] completed with state |
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by qa@nvidia.com)
Target branch:
mainJenkins build: https://prod.blsm.nvidia.com/swqa-tensorrt-qa-test/job/LLM_FUNCTION_CLUSTER_TEST/1485/
Bug(s): 6059036, 6181383, 6185150, 6211441, 6223556, 6241842, 6241845, 6245279, 6245765
Waive entries added
Already waived (skipped)
accuracy/test_llm_api_autodeploy.py::TestModelRegistryAccuracy::test_autodeploy_from_registry[nvidia_Llama-3.1-8B-Instruct-NVFP4-True]This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.
Summary by CodeRabbit