[None][infra] Waive 2 failed cases for main in pre-merge 47854

[trtllm-agent]

Auto-generated Waive PR

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

Waive entries added

unittest/_torch/modeling -k "modeling_out_of_tree" SKIP (https://nvbugs/6426847)
unittest/_torch/modeling/test_modeling_out_of_tree.py::TestOutOfTree::test_llm_api[True] SKIP (https://nvbugs/6426847)

---

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

Summary by CodeRabbit

• Tests
  • Expanded integration-test waiver coverage for out-of-tree modeling scenarios.
  • Added targeted handling for a specific LLM API test case and reorganized related waiver entries.
  • No changes to exported functionality or end-user behavior.

[jiaganc]

/bot skip --comment "waive only"

[coderabbitai]

📝 Walkthrough

Walkthrough

Expanded the integration waiver list for unittest/_torch/modeling with out-of-tree modeling skips and retained the existing Qwen waiver.

Changes

Integration waiver updates

Layer / File(s)	Summary
Modeling test waiver entries tests/integration/test_lists/waives.txt	Adds modeling_out_of_tree and TestOutOfTree::test_llm_api[True] skip entries while retaining the modeling_qwen waiver.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

• NVIDIA/TensorRT-LLM#15585: Adjusts related unittest/llmapi waiver entries in the same file.
• NVIDIA/TensorRT-LLM#16400: Updates other unittest waiver entries in the same file.
• NVIDIA/TensorRT-LLM#16412: Adds related test_llm_api waiver entries.

Suggested reviewers: dongxuy04, brnguyen2, bowenfu

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description has useful details, but it does not follow the required template sections for Description, Test Coverage, and PR Checklist.	Add the template sections with a short issue summary, test coverage, and checklist items, or use the required auto-generated format.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title is specific and matches the waive PR, including the infra type and affected pre-merge cases.
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.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@tests/integration/test_lists/waives.txt`:
- Around line 441-443: Add the repository-standard NVIDIA copyright header at
the beginning of tests/integration/test_lists/waives.txt, including the current
copyright year, while preserving the existing waiver entries unchanged.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 383c183c-9391-4bea-b773-362c7febff29

📥 Commits

Reviewing files that changed from the base of the PR and between e54fe55 and d120f09.

📒 Files selected for processing (1)

• tests/integration/test_lists/waives.txt

[tensorrt-cicd]

PR_Github #59440 [ skip ] triggered by Bot. Commit: d120f09 Link to invocation

[tensorrt-cicd]

PR_Github #59440 [ skip ] completed with state SUCCESS. Commit: d120f09
Skipping testing for commit d120f09

Link to invocation