Skip to content

[None][infra] Waive 4 failed cases for main in pre-merge 35639#13461

Merged
VALLIS-NERIA merged 1 commit intoNVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260425-131258
Apr 25, 2026
Merged

[None][infra] Waive 4 failed cases for main in pre-merge 35639#13461
VALLIS-NERIA merged 1 commit intoNVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260425-131258

Conversation

@ZhanruiSunCh
Copy link
Copy Markdown
Collaborator

@ZhanruiSunCh ZhanruiSunCh commented Apr 25, 2026

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by @VALLIS-NERIA)
Target branch: main
Bug(s): 5596343, 6011317, 6113016, 6113021

Waive entries added

accuracy/test_llm_api_pytorch.py::TestGPTOSS::test_eagle3_4gpus[v2_kv_cache-trtllm-one_model-overlap_scheduler] SKIP (https://nvbugs/6113016)
disaggregated/test_disaggregated.py::test_disaggregated_gpt_oss_120b_harmony[gpt_oss/gpt-oss-120b] SKIP (https://nvbugs/6011317)
accuracy/test_llm_api_pytorch.py::TestGPTOSS::test_w4_4gpus[v2_kv_cache-dp4-cutlass-auto] SKIP (https://nvbugs/5596343)
unittest/_torch/speculative/test_eagle3.py::test_llama_eagle3_dynamic_tree[True-False] SKIP (https://nvbugs/6113021)

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

Summary by CodeRabbit

  • Tests
    • Updated the test skip list to exclude additional test cases requiring follow-up work.

Bug(s): 5596343, 6011317, 6113016, 6113021
Requested by: @VALLIS-NERIA

Signed-off-by: ZhanruiSunCh <184402041+ZhanruiSunCh@users.noreply.github.com>
@VALLIS-NERIA
Copy link
Copy Markdown
Collaborator

/bot skip --comment "clean pre merge"

@VALLIS-NERIA VALLIS-NERIA enabled auto-merge (squash) April 25, 2026 13:14
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 11828f16-8b21-413c-bef6-97ada671c746

📥 Commits

Reviewing files that changed from the base of the PR and between ce62383 and 0c23966.

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

📝 Walkthrough

Walkthrough

This pull request adds four new test skip entries to the waivers list file, each marking specific test parametrizations to be excluded from execution and associating them with corresponding nvbugs tickets for tracking.

Changes

Cohort / File(s) Summary
Test Waivers
tests/integration/test_lists/waives.txt
Added four new skipped test entries: test_eagle3_4gpus with v2_kv_cache-trtllm-one_model-overlap_scheduler, test_disaggregated_gpt_oss_120b_harmony with gpt_oss/gpt-oss-120b, test_w4_4gpus with v2_kv_cache-dp4-cutlass-auto, and test_llama_eagle3_dynamic_tree with True-False, each annotated with associated nvbugs URLs.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • VALLIS-NERIA
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly follows the template format and accurately describes the PR's purpose: waiving 4 failed test cases with the associated bug tracking identifier.
Description check ✅ Passed The PR description is well-structured with clear identification of the bug references, specific waive entries with test paths and bug URLs, and appropriate disclaimers about the auto-generated nature.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #45504 [ skip ] triggered by Bot. Commit: 0c23966 Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #45504 [ skip ] completed with state SUCCESS. Commit: 0c23966
Skipping testing for commit 0c23966

Link to invocation

@VALLIS-NERIA VALLIS-NERIA merged commit dc8de93 into NVIDIA:main Apr 25, 2026
8 checks passed
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