Skip to content

[None][infra] Waive 5 failed cases for main in post-merge 2599#12283

Merged
EmmaQiaoCh merged 2 commits intoNVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260317-130122
Mar 17, 2026
Merged

[None][infra] Waive 5 failed cases for main in post-merge 2599#12283
EmmaQiaoCh merged 2 commits intoNVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260317-130122

Conversation

@ZhanruiSunCh
Copy link
Collaborator

@ZhanruiSunCh ZhanruiSunCh commented Mar 17, 2026

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by @EmmaQiaoCh)
Target branch: main
Bug(s): 5989907, 5989912, 5989920, 5989923

Waive entries added

test_e2e.py::test_draft_token_tree_quickstart_advanced_eagle3_depth_1_tree[Llama-3.1-8b-Instruct-llama-3.1-model/Llama-3.1-8B-Instruct-EAGLE3-LLaMA3.1-Instruct-8B] SKIP (https://nvbugs/5989907)
test_e2e.py::test_draft_token_tree_quickstart_advanced_eagle3[Llama-3.1-8b-Instruct-llama-3.1-model/Llama-3.1-8B-Instruct-EAGLE3-LLaMA3.1-Instruct-8B] SKIP (https://nvbugs/5989907)
unittest/_torch/modules/moe/test_moe_backend.py::test_moe_backend -k "TRTLLM" SKIP (https://nvbugs/5989912)
accuracy/test_llm_api_pytorch.py::TestDeepSeekV32::test_nvfp4_multi_gpus_piecewise_cuda_graph[mtp3_fp8kv_chunked] SKIP (https://nvbugs/5989920)
unittest/disaggregated/test_py_cache_transceiver_mp.py::test_v2_transceiver_mp[ctx_first-v2_mp_tp1_pp2_to_tp1_pp2] SKIP (https://nvbugs/5989923)

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

Summary by CodeRabbit

  • Tests
    • Added test skip entries to manage test execution configuration.

Bug(s): 5989907, 5989912, 5989920, 5989923
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 17, 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: 5497d66d-f030-4589-bbd3-2c54b84f2fa3

📥 Commits

Reviewing files that changed from the base of the PR and between 402a056 and 31b1c07.

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

📝 Walkthrough

Walkthrough

Five new SKIP entries added to the test waives file, addressing two draft token tree quickstart tests and three MOE/NVBUG-related tests. No functional code changes; configuration-only modification to test collection behavior.

Changes

Cohort / File(s) Summary
Test Waives Configuration
tests/integration/test_lists/waives.txt
Added five SKIP entries for draft token tree quickstart tests and MOE/NVBUG-related tests.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

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

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description includes waive entries with bug references and notes auto-generation, but lacks a proper Description section explaining the issue/solution and Test Coverage section as required by the template. Add a brief Description section explaining why these test cases are being waived and a Test Coverage section documenting which tests validate the waive entries.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: adding waive entries for 5 failed test cases, with clear context about the infra type and post-merge context.
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
📝 Coding Plan
  • Generate coding plan for human review comments

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

Tip

You can enable review details to help with troubleshooting, context usage and more.

Enable the reviews.review_details setting to include review details such as the model used, the time taken for each step and more in the review comments.

@EmmaQiaoCh
Copy link
Collaborator

/bot skip --comment "Waive failed cases"

@EmmaQiaoCh EmmaQiaoCh enabled auto-merge (squash) March 17, 2026 13:03
@tensorrt-cicd
Copy link
Collaborator

PR_Github #39257 [ skip ] triggered by Bot. Commit: 31b1c07 Link to invocation

@tensorrt-cicd
Copy link
Collaborator

PR_Github #39257 [ skip ] completed with state SUCCESS. Commit: 31b1c07
Skipping testing for commit 31b1c07

Link to invocation

@EmmaQiaoCh EmmaQiaoCh merged commit 4c4b688 into NVIDIA:main Mar 17, 2026
7 checks passed
limin2021 pushed a commit to limin2021/TensorRT-LLM that referenced this pull request Mar 19, 2026
…A#12283)

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