Skip to content

[None][infra] Waive 1 failed cases for main in pre-merge 39661 - #14518

Closed
ZhanruiSunCh wants to merge 1 commit into
NVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260525-065037
Closed

[None][infra] Waive 1 failed cases for main in pre-merge 39661#14518
ZhanruiSunCh wants to merge 1 commit into
NVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260525-065037

Conversation

@ZhanruiSunCh

@ZhanruiSunCh ZhanruiSunCh commented May 25, 2026

Copy link
Copy Markdown
Collaborator

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by @chienchunhung)
Target branch: main
Bug(s): 6216097

Waive entries added

full:DGX_B200/accuracy/test_disaggregated_serving.py::TestDeepSeekV3Lite::test_auto_dtype_with_helix[fifo_v2-cudagraph:with_padding-pp1dp2cp2] SKIP (https://nvbugs/6216097)

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

Summary by CodeRabbit

  • Tests
    • Added test configuration to skip a specific test scenario to address a known issue.

Review Change Stack

Bug(s): 6216097
Requested by: @chienchunhung

Signed-off-by: ZhanruiSunCh <184402041+ZhanruiSunCh@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented May 25, 2026

Copy link
Copy Markdown
Contributor

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: d48cf7fa-14c9-4e4e-bbe4-48038353e4af

📥 Commits

Reviewing files that changed from the base of the PR and between e6d4f9f and 5263d7f.

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

📝 Walkthrough

Walkthrough

This PR adds a single SKIP waiver entry to the test exclusion list, marking a specific DGX B200 DeepSeekV3Lite auto-dtype test parameterization as excluded from the test suite, with a reference to a tracked bug.

Changes

Test waiver update

Layer / File(s) Summary
Test waiver entry for auto-dtype test
tests/integration/test_lists/waives.txt
Added a SKIP waiver for TestDeepSeekV3Lite::test_auto_dtype_with_helix on DGX B200 with fifo_v2-cudagraph:with_padding-pp1dp2cp2 configuration, linked to nvbugs/6216097.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Possibly related PRs

  • NVIDIA/TensorRT-LLM#14514: Also adds SKIP entries to tests/integration/test_lists/waives.txt including an nvbugs/6216097-related waiver.
  • NVIDIA/TensorRT-LLM#14504: Adds waiver entries for other test_auto_dtype test cases to the same waiver list.
  • NVIDIA/TensorRT-LLM#14034: Adds similar SKIP waiver entries to tests/integration/test_lists/waives.txt for other failing tests.

Suggested reviewers

  • yiqingy0
  • jieli-matrix
  • EmmaQiaoCh
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies this as an infrastructure change waiving failed test cases, with the specific count (1) and issue reference (pre-merge 39661), making it specific and informative for repository history.
Description check ✅ Passed The description includes the auto-generated nature, clear bug reference (6216097), specific waive entry added, and appropriately asks for review before merging, meeting the essential information requirements.
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.

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.

2 participants