[None][test] Waive 11 failed cases for main in QA CI - #15506
Conversation
📝 WalkthroughWalkthroughAdds 11 new SKIP waiver lines to ChangesGB300 Waiver Additions
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: 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 209-226: Remove all duplicate waive entries in the waives.txt file
where the same test nodeid and nvbug reference appear multiple times.
Specifically, remove one instance of each duplicate pair:
TestNemotronNanoV3::test_accuracy[nvfp4-1-attn_dp_off-trtllm],
TestNemotronSuperV3::test_mtp[bf16_ws4_180gb-trtllm],
TestNemotronSuperV3::test_mtp[fp8_ws4_80gb-trtllm],
TestNemotronSuperV3::test_mtp[nvfp4_ws4_80gb-trtllm],
TestGPTOSS::test_w4_4gpus[v2_kv_cache-ep4-trtllm-fp8], and
TestGPTOSS::test_w4_4gpus[v2_kv_cache_no_reuse-tp4-trtllm-fp8]. Additionally,
consolidate entries that have the same test nodeid but different nvbug
references by keeping only one entry with the most recent or correct nvbug
reference. For TestQwen3_5_35B_A3B::test_fp8_moe_dflash and
TestQwen2_5_VL_7B::test_auto_dtype, determine which nvbug reference is correct
and remove the redundant entry. Ensure the file maintains one entry per unique
test nodeid to pass the waive list check.
🪄 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: 278b275a-95e2-48d8-baa0-c2d4666da041
📒 Files selected for processing (1)
tests/integration/test_lists/waives.txt
45c069e to
335ebe5
Compare
335ebe5 to
e31cd9c
Compare
Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com>
Bug(s): 6316980, 6316981, 6317600, 6329165, 6331421, 6344883, 6344884 Requested by: qa@nvidia.com Jenkins build: https://prod.blsm.nvidia.com/swqa-tensorrt-qa-test/job/LLM_FUNCTION_CLUSTER_TEST/1555/ Signed-off-by: tensorrt-cicd <90828364+tensorrt-cicd@users.noreply.github.com> Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com>
Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com>
84a070f to
e019f69
Compare
|
/bot skip --comment "Only waive tests" |
|
PR_Github #55145 [ skip ] triggered by Bot. Commit: |
|
PR_Github #55145 [ skip ] completed with state |
Signed-off-by: tensorrt-cicd <90828364+tensorrt-cicd@users.noreply.github.com> Co-authored-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com> Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com>
Signed-off-by: tensorrt-cicd <90828364+tensorrt-cicd@users.noreply.github.com> Co-authored-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
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/1555/
Bug(s): 6316980, 6316981, 6317600, 6329165, 6331421, 6344883, 6344884
Waive entries added
This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.
Summary by CodeRabbit
Release Notes
This release contains internal testing infrastructure updates only. No user-facing features, bug fixes, or changes have been introduced.