Skip to content

[None][infra] Waive 1 failed cases for main in pre-merge 38987#14350

Merged
tcherckez-nvidia merged 1 commit into
NVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260520-085202
May 20, 2026
Merged

[None][infra] Waive 1 failed cases for main in pre-merge 38987#14350
tcherckez-nvidia merged 1 commit into
NVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260520-085202

Conversation

@ZhanruiSunCh
Copy link
Copy Markdown
Collaborator

@ZhanruiSunCh ZhanruiSunCh commented May 20, 2026 •
edited by coderabbitai Bot
Loading

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by @tcherckez-nvidia)
Target branch: main
Bug(s): 6194934

Waive entries added

accuracy/test_llm_api_autodeploy.py::TestGemmaE2B::test_gemma4_e2b_it SKIP (https://nvbugs/6194934)

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

Summary by CodeRabbit

  • Tests
    • A test case has been added to the known issues list and will be skipped during automated testing while a related issue is being addressed.

Review Change Stack

@tcherckez-nvidia
Copy link
Copy Markdown
Collaborator

tcherckez-nvidia commented May 20, 2026

/bot run

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 20, 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: 388a8b63-d474-45ba-aebb-6d2a362aeb77

📥 Commits

Reviewing files that changed from the base of the PR and between 9e3dbaa and 546b6ed.

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

Walkthrough

A single test waiver entry is added to skip TestGemmaE2B::test_gemma4_e2b_it from the autodeploy accuracy test suite, with an associated nvbug reference to track the underlying issue.

Changes

Test Waiver Update

Layer / File(s) Summary
Add GemmaE2B autodeploy test waiver
tests/integration/test_lists/waives.txt		 A single waiver entry is added to skip the accuracy/test_llm_api_autodeploy.py::TestGemmaE2B::test_gemma4_e2b_it test case and link it to nvbug 6194934.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title follows the required template format with [None] ticket, [infra] type, and clearly describes the main change: adding a waiver for a failed test case in the pre-merge process.
Description check ✅ Passed The description is auto-generated but provides essential information: the waive entry added, associated bug ID, and context about the auto-generation. However, it lacks the standard PR description sections (Description, Test Coverage) and PR checklist from the template.
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

tensorrt-cicd commented May 20, 2026

PR_Github #49400 [ run ] triggered by Bot. Commit: 546b6ed Link to invocation

@tcherckez-nvidia tcherckez-nvidia enabled auto-merge (squash) May 20, 2026 10:28
@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

tensorrt-cicd commented May 20, 2026

PR_Github #49400 [ run ] completed with state SUCCESS. Commit: 546b6ed
/LLM/main/L0_MergeRequest_PR pipeline #39049 completed with status: 'SUCCESS'

CI Report

Link to invocation

@tcherckez-nvidia tcherckez-nvidia merged commit b8f78a1 into NVIDIA:main May 20, 2026
12 checks passed
xxi-nv pushed a commit to xxi-nv/TensorRT-LLM that referenced this pull request May 22, 2026
@ZhanruiSunCh @xxi-nv
[None][infra] Waive 1 failed cases for main in pre-merge 38987 (NVIDI…
4ce16ca 
…A#14350)

Signed-off-by: ZhanruiSunCh <[email protected]>
@coderabbitai coderabbitai Bot mentioned this pull request May 25, 2026
Closed
bmarimuthu-nv pushed a commit to nv-auto-deploy/TensorRT-LLM that referenced this pull request May 28, 2026
@ZhanruiSunCh @bmarimuthu-nv
[None][infra] Waive 1 failed cases for main in pre-merge 38987 (NVIDI…
2f1f046 
…A#14350)

Signed-off-by: ZhanruiSunCh <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tcherckez-nvidia tcherckez-nvidia tcherckez-nvidia approved these changes

Assignees

@ZhanruiSunCh ZhanruiSunCh

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ZhanruiSunCh @tcherckez-nvidia @tensorrt-cicd