Skip to content

[None][infra] Waive 5 failed cases for main in post-merge 2578#12023

Merged
EmmaQiaoCh merged 2 commits intoNVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260309-053754
Mar 9, 2026
Merged

[None][infra] Waive 5 failed cases for main in post-merge 2578#12023
EmmaQiaoCh merged 2 commits intoNVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260309-053754

Conversation

@ZhanruiSunCh
Copy link
Collaborator

@ZhanruiSunCh ZhanruiSunCh commented Mar 9, 2026

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by @EmmaQiaoCh)
Target branch: main
Bug(s): 4846166, 5846166, 5961736, 5961739, 5961814

Waive entries added

accuracy/test_disaggregated_serving.py::TestDeepSeekV32Exp::test_auto_dtype[False] SKIP (https://nvbugs/5961736)
unittest/auto_deploy/multigpu/transformations/library/test_tp_sharding.py::test_sharding[Linear-torch_dist_all_gather-True-False-2] SKIP (https://nvbugs/5961739)
perf/test_perf_sanity.py::test_e2e[disagg_upload-gen_only-gb200_gpt-oss-120b-fp4_1k1k_con64_ctx1_tp1_gen1_tp4_eplb0_mtp0_ccb-UCX] SKIP (https://nvbugs/4846166)
perf/test_perf_sanity.py::test_e2e[disagg_upload-gen_only-gb200_deepseek-r1-fp4_1k1k_con3072_ctx1_dep4_gen1_dep4_eplb0_mtp1_ccb-UCX] SKIP (https://nvbugs/5846166)
accuracy/test_llm_api_pytorch.py::TestDeepSeekV3Lite::test_nvfp4_4gpus[moe_backend=CUTLASS-mtp_nextn=0-ep4-fp8kv=True-attention_dp=True-cuda_graph=True-overlap_scheduler=True-torch_compile=True] SKIP (https://nvbugs/5961814)

This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.

Summary by CodeRabbit

  • Tests
    • Updated test skip list with additional entries to mark specific test cases for skipping.

Bug(s): 4846166, 5846166, 5961736, 5961739, 5961814
Requested by: @EmmaQiaoCh

Signed-off-by: ZhanruiSunCh <184402041+ZhanruiSunCh@users.noreply.github.com>
Signed-off-by: Emma Qiao <qqiao@nvidia.com>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 9, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f01775f1-2edf-4d21-a7c5-7598defeec91

📥 Commits

Reviewing files that changed from the base of the PR and between 91233d5 and df7c359.

📒 Files selected for processing (1)
  • tests/integration/test_lists/waives.txt

📝 Walkthrough

Walkthrough

This PR adds five new test case entries to the test waives list, marking specific test paths as SKIP with associated references to NV bug tracking or notes.

Changes

Cohort / File(s) Summary
Test Waives Configuration
tests/integration/test_lists/waives.txt
Added five new SKIP entries for test cases with associated bug/note URLs.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • chzblych
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is incomplete. It lacks required sections: no explicit 'Description' or 'Test Coverage' sections that explain the issue and solution, and the PR Checklist items are not reviewed, which are mandatory per the template. Add explicit 'Description' section explaining why these tests are being waived and 'Test Coverage' section. Complete the PR Checklist by reviewing and checking off applicable items as specified in the template.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title '[None][infra] Waive 5 failed cases for main in post-merge 2578' clearly summarizes the main change: adding waive entries for 5 failed test cases, following the repository's required format with ticket reference, type, and purpose.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Comment @coderabbitai help to get the list of available commands and usage tips.

@EmmaQiaoCh
Copy link
Collaborator

/bot skip --comment "Waive failed cases"

@EmmaQiaoCh EmmaQiaoCh enabled auto-merge (squash) March 9, 2026 05:41
@tensorrt-cicd
Copy link
Collaborator

PR_Github #38206 [ skip ] triggered by Bot. Commit: df7c359 Link to invocation

@tensorrt-cicd
Copy link
Collaborator

PR_Github #38206 [ skip ] completed with state SUCCESS. Commit: df7c359
Skipping testing for commit df7c359

Link to invocation

@EmmaQiaoCh EmmaQiaoCh merged commit d17046d into NVIDIA:main Mar 9, 2026
7 checks passed
perf/test_perf_sanity.py::test_e2e[disagg_upload-gen_only-gb200_deepseek-v32-fp4_1k1k_con2048_ctx1_dep4_gen1_dep4_eplb0_mtp1_ccb-UCX] SKIP (https://nvbugs/5846166)
accuracy/test_disaggregated_serving.py::TestDeepSeekV32Exp::test_auto_dtype[False] SKIP (https://nvbugs/5961736)
unittest/auto_deploy/multigpu/transformations/library/test_tp_sharding.py::test_sharding[Linear-torch_dist_all_gather-True-False-2] SKIP (https://nvbugs/5961739)
perf/test_perf_sanity.py::test_e2e[disagg_upload-gen_only-gb200_gpt-oss-120b-fp4_1k1k_con64_ctx1_tp1_gen1_tp4_eplb0_mtp0_ccb-UCX] SKIP (https://nvbugs/4846166)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be 5846166?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh...yes!, My typo when link the nvbug. I'll correct it in next waive pr.
Thanks a lot!

tianyuz-nv pushed a commit to wanqian-nv/TensorRT-LLM that referenced this pull request Mar 19, 2026
…A#12023)

Signed-off-by: ZhanruiSunCh <184402041+ZhanruiSunCh@users.noreply.github.com>
Signed-off-by: Emma Qiao <qqiao@nvidia.com>
Co-authored-by: Emma Qiao <qqiao@nvidia.com>
limin2021 pushed a commit to limin2021/TensorRT-LLM that referenced this pull request Mar 19, 2026
…A#12023)

Signed-off-by: ZhanruiSunCh <184402041+ZhanruiSunCh@users.noreply.github.com>
Signed-off-by: Emma Qiao <qqiao@nvidia.com>
Co-authored-by: Emma Qiao <qqiao@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants