Skip to content

[None][infra] Waive 2 failed cases for main in post-merge 2709#13840

Merged
mzweilz merged 1 commit into
NVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260507-070544
May 7, 2026
Merged

[None][infra] Waive 2 failed cases for main in post-merge 2709#13840
mzweilz merged 1 commit into
NVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260507-070544

Conversation

@ZhanruiSunCh
Copy link
Copy Markdown
Collaborator

@ZhanruiSunCh ZhanruiSunCh commented May 7, 2026

Auto-generated Waive PR

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

Waive entries added

unittest/llmapi/test_llm_pytorch.py::test_nemotron_nas_lora[None] SKIP (https://nvbugs/6140399)
unittest/llmapi/test_llm_pytorch.py::test_nemotron_nas_lora[cuda_graph_config0] SKIP (https://nvbugs/6140399)

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

Summary by CodeRabbit

  • Tests
    • Updated test waiver list to accommodate additional test cases for LLM PyTorch integration scenarios.

Bug(s): 6140399
Requested by: @mzweilz

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

coderabbitai Bot commented May 7, 2026

Review Change Stack
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: e5171aba-b7b9-4f4c-8a62-d79edc5c8895

📥 Commits

Reviewing files that changed from the base of the PR and between 2ed3dcb and 5248bd7.

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

📝 Walkthrough

Walkthrough

Two test waiver entries added to the exclusion list for test_nemotron_nas_lora in unittest/llmapi/test_llm_pytorch.py, covering parameter combinations None and cuda_graph_config0, both marked SKIP with reference nvbugs/6140399.

Changes

Test Waiver Updates

Layer / File(s) Summary
Test Waiver Entries
tests/integration/test_lists/waives.txt
Waiver entries added for test_nemotron_nas_lora test cases with parameters None and cuda_graph_config0, both referencing nvbugs/6140399.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • mzweilz
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: waiving 2 failed test cases for the main branch in post-merge run 2709.
Description check ✅ Passed The PR description explains the purpose (auto-generated waive PR), lists the specific test cases being waived, and references the associated bug. However, it deviates from the template by not including explicit sections for Description, Test Coverage, and PR Checklist.
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

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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

@mzweilz
Copy link
Copy Markdown
Collaborator

mzweilz commented May 7, 2026

/bot run --stage-list "A10-Build_Docs"

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #47151 [ run ] triggered by Bot. Commit: 5248bd7 Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #47151 [ run ] completed with state SUCCESS. Commit: 5248bd7
/LLM/main/L0_MergeRequest_PR pipeline #37112 (Partly Tested) completed with status: 'SUCCESS'

CI Report

Link to invocation

@mzweilz
Copy link
Copy Markdown
Collaborator

mzweilz commented May 7, 2026

/bot skip --comment "Test list check passed."

@mzweilz mzweilz enabled auto-merge (squash) May 7, 2026 08:54
@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #47171 [ skip ] triggered by Bot. Commit: 5248bd7 Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #47171 [ skip ] completed with state SUCCESS. Commit: 5248bd7
Skipping testing for commit 5248bd7

Link to invocation

@mzweilz mzweilz merged commit 9d9f817 into NVIDIA:main May 7, 2026
10 checks passed
yufeiwu-nv pushed a commit to yufeiwu-nv/TensorRT-LLM that referenced this pull request May 19, 2026
…A#13840)

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