Skip to content

[None][infra] Waive 3 failed cases for main in post-merge 2566#11881

Merged
EmmaQiaoCh merged 2 commits intoNVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260304-021346
Mar 4, 2026
Merged

[None][infra] Waive 3 failed cases for main in post-merge 2566#11881
EmmaQiaoCh merged 2 commits intoNVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260304-021346

Conversation

@ZhanruiSunCh
Copy link
Collaborator

@ZhanruiSunCh ZhanruiSunCh commented Mar 4, 2026

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by @EmmaQiaoCh)
Target branch: main
Bug(s): 5948428, 5948435

Waive entries added

accuracy/test_disaggregated_serving.py::TestDeepSeekV3Lite::test_guided_decoding[llguidance-mtp_nextn=2] SKIP (https://nvbugs/5948428)
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=False] SKIP (https://nvbugs/5948435)
accuracy/test_disaggregated_serving.py::TestDeepSeekV3Lite::test_guided_decoding[llguidance-mtp_nextn=0] SKIP (https://nvbugs/5948428)

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

Summary by CodeRabbit

  • Tests
    • Updated internal test configuration for guided decoding scenarios.

Note: This release contains no user-visible changes. Updates are limited to test infrastructure and internal validation processes.

Bug(s): 5948428, 5948435
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 4, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 58a396f7-eeb1-4a86-aeeb-37aa177298fb

📥 Commits

Reviewing files that changed from the base of the PR and between 0a5a5e7 and bf2c1ac.

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

📝 Walkthrough

Walkthrough

The PR adds three new SKIP entries to the test waives list for DeepSeekV3Lite guided decoding tests in disaggregated scenarios with various mtp_nextn and GPU configurations.

Changes

Cohort / File(s) Summary
Test Waives Configuration
tests/integration/test_lists/waives.txt
Added 3 new SKIP entries for DeepSeekV3Lite guided decoding tests in disaggregated contexts, covering different mtp_nextn configurations and nvfp4_4gpus variant in LLM API PyTorch tests.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

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

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description includes the key information about the auto-generated waive entries with linked bug references, but does not follow the required template structure with explicit Description, Test Coverage, and PR Checklist sections. While the description contains essential details, consider reformatting it to explicitly align with the template sections (Description, Test Coverage, PR Checklist) for consistency with repository standards.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title '[None][infra] Waive 3 failed cases for main in post-merge 2566' directly summarizes the main change: adding three waive entries to skip failing test cases.
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 for main"

@EmmaQiaoCh EmmaQiaoCh enabled auto-merge (squash) March 4, 2026 02:16
@tensorrt-cicd
Copy link
Collaborator

PR_Github #37592 [ skip ] triggered by Bot. Commit: bf2c1ac Link to invocation

@tensorrt-cicd
Copy link
Collaborator

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

Link to invocation

@EmmaQiaoCh EmmaQiaoCh merged commit 77d48e0 into NVIDIA:main Mar 4, 2026
7 checks passed
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Mar 9, 2026
…A#11881)

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>
tianyuz-nv pushed a commit to wanqian-nv/TensorRT-LLM that referenced this pull request Mar 19, 2026
…A#11881)

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#11881)

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.

3 participants