[None][test] Waive 6 failed cases for main in QA CI - #14787
Conversation
Bug(s): 6185150, 6211193, 6223556, 6245098 Requested by: qa@nvidia.com Jenkins build: https://prod.blsm.nvidia.com/swqa-tensorrt-qa-test/job/LLM_FUNCTION_CLUSTER_TEST/1482/ 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 |
|
PR_Github #51327 [ run ] triggered by Bot. Commit: |
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`:
- Line 118: Add a waiver entry for the DeepSeek-V3 failure in
tests/integration/test_lists/waives.txt for the test named
test_e2e.py::test_ptp_quickstart_advanced_deepseek_multi_nodes[DeepSeek-V3] that
references https://nvbugs/6245098 (or alternatively update the PR summary to
remove the claim that 6245098 is waived); ensure the new line follows the
existing waive list format and includes the exact test identifier and the nvbugs
URL so the waiver is recognized.
🪄 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: db2d8b90-3a6b-467b-abc2-8a17a079d954
📒 Files selected for processing (1)
tests/integration/test_lists/waives.txt
📝 WalkthroughWalkthroughThis PR adds 5 new SKIP waiver entries to the integration test skip list. The entries cover failing test cases: Qwen3NextInstruct BF16 with tensor parallelism and cuGraphs, Qwen3 FP8, two TinyLlama disaggregated overlap generation variants, and OpenAI disaggregated multi-node completion service discovery with etcd. ChangesTest Skip Waivers
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 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.
♻️ Duplicate comments (1)
tests/integration/test_lists/waives.txt (1)
118-118:⚠️ Potential issue | 🟡 Minor | ⚡ Quick winPR summary claims 6 waivers but only 5 entries are present
The PR summary states that 6 test cases are being waived, but this file contains only 5 new SKIP entries. The missing case
test_e2e.py::test_ptp_quickstart_advanced_deepseek_multi_nodes[DeepSeek-V3](https://nvbugs/6245098) referenced in the summary is not present in the waives.txt file. Either add the missing waiver entry or update the PR summary to accurately reflect 5 waivers.🤖 Prompt for 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. In `@tests/integration/test_lists/waives.txt` at line 118, PR summary claims 6 waived tests but only 5 SKIP entries exist in the waives list; either add the missing waiver for the test_e2e.py::test_ptp_quickstart_advanced_deepseek_multi_nodes[DeepSeek-V3] (link nvbugs/6245098) to the waives file or change the PR summary to state 5 waivers; update the waives file by adding the exact SKIP line for test_e2e.py::test_ptp_quickstart_advanced_deepseek_multi_nodes[DeepSeek-V3] referencing the bug URL, or if that waiver is not needed remove the extra count in the PR summary so it matches the current entries.
🤖 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.
Duplicate comments:
In `@tests/integration/test_lists/waives.txt`:
- Line 118: PR summary claims 6 waived tests but only 5 SKIP entries exist in
the waives list; either add the missing waiver for the
test_e2e.py::test_ptp_quickstart_advanced_deepseek_multi_nodes[DeepSeek-V3]
(link nvbugs/6245098) to the waives file or change the PR summary to state 5
waivers; update the waives file by adding the exact SKIP line for
test_e2e.py::test_ptp_quickstart_advanced_deepseek_multi_nodes[DeepSeek-V3]
referencing the bug URL, or if that waiver is not needed remove the extra count
in the PR summary so it matches the current entries.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: a4ed7824-1b18-4ef1-876b-a1b7813d8c65
📒 Files selected for processing (1)
tests/integration/test_lists/waives.txt
|
PR_Github #51327 [ run ] completed with state
|
|
/bot run --skip-test |
|
/bot run --skip-test |
|
PR_Github #51465 [ run ] triggered by Bot. Commit: |
Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
|
PR_Github #51465 [ run ] completed with state |
|
/bot reuse-pipeline |
|
PR_Github #51510 [ reuse-pipeline ] triggered by Bot. Commit: |
|
/bot reuse-pipeline |
|
PR_Github #51515 [ reuse-pipeline ] triggered by Bot. Commit: |
|
PR_Github #51510 [ reuse-pipeline ] completed with state |
|
PR_Github #51515 [ 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/1482/
Bug(s): 6185150, 6211193, 6223556, 6245098
Waive entries added
This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.
Summary by CodeRabbit
Note: This release contains no user-visible changes—updates are limited to internal testing infrastructure only.