Skip to content

[None][infra] Waive 3 failed cases for main in pre-merge 37379#13941

Merged
Hudayday merged 1 commit into
NVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260509-114523
May 9, 2026
Merged

[None][infra] Waive 3 failed cases for main in pre-merge 37379#13941
Hudayday merged 1 commit into
NVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260509-114523

Conversation

@ZhanruiSunCh
Copy link
Copy Markdown
Collaborator

@ZhanruiSunCh ZhanruiSunCh commented May 9, 2026

Auto-generated Waive PR

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

Waive entries added

unittest/auto_deploy/singlegpu/transformations SKIP (https://nvbugs/6160629)
unittest/auto_deploy/standalone SKIP (https://nvbugs/6160629)
unittest/auto_deploy/standalone/test_standalone_package.py::TestStandalonePackage::test_run_unit_tests SKIP (https://nvbugs/6160629)

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

Summary by CodeRabbit

  • Chores
    • Updated test infrastructure and automated deployment waiver configurations to expand coverage for specific testing scenarios.

Review Change Stack

Bug(s): 6160629
Requested by: @Hudayday

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

coderabbitai Bot commented May 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: Enterprise

Run ID: 94ce0440-c94d-48dd-bf9e-4f782d5b3660

📥 Commits

Reviewing files that changed from the base of the PR and between f8d7ecb and 90021e1.

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

📝 Walkthrough

Walkthrough

The PR extends test waiver coverage in the integration test waiver list by adding new SKIP entries for the unittest/auto_deploy/singlegpu/transformations directory and the unittest/auto_deploy/standalone module, while preserving existing waivers for library-level transformations tests.

Changes

Auto-Deploy Test Waiver Coverage

Layer / File(s) Summary
Test Waiver Updates
tests/integration/test_lists/waives.txt
Three new SKIP entries added to the auto-deploy section: covering singlegpu/transformations, standalone, and the specific TestStandalonePackage::test_run_unit_tests test case. Existing singlegpu/transformations/library waivers are retained and repositioned.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • NVIDIA/TensorRT-LLM#13648: Adds waiver for auto_deploy/standalone unittest; this PR extends that change with additional singlegpu/transformations waivers.

Suggested reviewers

  • achartier
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description explains the auto-generated nature of the PR, references the associated bug, lists the specific waive entries added, and includes context about the request. However, it does not follow the repository's required description template with sections like 'Description', 'Test Coverage', and 'PR Checklist'. Consider whether this auto-generated PR format is an exception to the template requirement, or if key template sections (especially the PR Checklist) should be included for consistency.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the PR as waiving 3 failed test cases for the main branch in a specific pre-merge run, which matches the changeset's purpose of adding SKIP entries to the waives.txt file.
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.

@Hudayday Hudayday self-requested a review May 9, 2026 11:46
@Hudayday Hudayday enabled auto-merge (squash) May 9, 2026 11:47
@Hudayday
Copy link
Copy Markdown
Collaborator

Hudayday commented May 9, 2026

/bot skip --comment "waive tests"

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #47514 [ skip ] triggered by Bot. Commit: 90021e1 Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #47514 [ skip ] completed with state SUCCESS. Commit: 90021e1
Skipping testing for commit 90021e1

Link to invocation

@Hudayday Hudayday merged commit aeb9895 into NVIDIA:main May 9, 2026
12 checks passed
yufeiwu-nv pushed a commit to yufeiwu-nv/TensorRT-LLM that referenced this pull request May 19, 2026
…A#13941)

Signed-off-by: ZhanruiSunCh <184402041+ZhanruiSunCh@users.noreply.github.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