[trtllm-agent]

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by @xinhe-nv)
Target branch: main
Bug(s): 6981910

Waive entries added

full:CPU/unittest/executor/test_forward_error_survival.py SKIP (https://nvbugs/6981910)

---

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

Dev Engineer Review

The change adds a SKIP waiver for full:CPU/unittest/executor/test_forward_error_survival.py, linked to nvbug 6981910. The scope is limited to the waiver list. The reported pre-merge failure is the stated reason for the waiver; no test result was supplied to verify its current status.

QA Engineer Review

Modified tests/integration/test_lists/waives.txt with one CPU test-suite waiver. No test code changed. Coverage verdict: needs follow-up; confirm the failure is resolved and remove or update the waiver.

Per-File QA Perspective

• tests/integration/test_lists/waives.txt: Adds a SKIP entry for full:CPU/unittest/executor/test_forward_error_survival.py. The test is waived rather than added to a CI or manual-QA list.

[xinhe-nv]

/bot skip --comment "waive failure tests"

[github-actions]

Automatically added "ci: full pre-merge approved" because this PR has satisfied the required GitHub review approvals. Unresolved review conversations and other required checks remain independent merge requirements.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Source mode: Auto
• Repository: NVIDIA/TensorRT-LLM
• Branch: main
• File: .coderabbit.yaml
• Commit: 41b0085
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Enterprise
• Run ID: 8f35816b-84da-4f2e-ab8f-e2052ccc2e0f

📥 Commits

Reviewing files that changed from the base of the PR and between 41b0085 and f18f0aa.

📒 Files selected for processing: 1

• tests/integration/test_lists/waives.txt

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 11 remain after this review.

Walkthrough

The CPU test skip list now waives full:CPU/unittest/executor/test_forward_error_survival.py and references nvbug 6981910.

Changes

CPU test waiver

Layer / File(s)	Summary
Add CPU test waiver tests/integration/test_lists/waives.txt	The skip list adds an entry for the forward error survival test and references nvbug 6981910.

Priority: ⬇️ Low

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

Change: Other

Possibly related PRs

• NVIDIA/TensorRT-LLM#20883: Updates test fakes in the waived test and adds fields used by its validation tests.

Suggested reviewers: brnguyen2

Merge Risk: ⚪ Minimal · up to f18f0

This targeted waiver is mergeable as submitted.

Pre-merge checks: 5

Check name	Status	Explanation
Title check		The title follows the repository's [ticket][type] Summary pattern and names the main change, a CI waive for one failed pre-merge case. It does not name the specific test file or bug, but it is not mis…
Description check		The description covers the auto-generated waive PR, the target branch (main), the bug link (nvbugs 6981910), and the waive entry itself. It does not fill in the template sections for Description, Test…
Docstring Coverage		No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Title check

Explanation

The title follows the repository's [ticket][type] Summary pattern and names the main change, a CI waive for one failed pre-merge case. It does not name the specific test file or bug, but it is not misleading. The bracket ticket field is [None], which fits a change with no ticket, while the bug ID appears only in the description. Title format and content are acceptable.

Full details: Description check

Explanation

The description covers the auto-generated waive PR, the target branch (main), the bug link (nvbugs 6981910), and the waive entry itself. It does not fill in the template sections for Description, Test Coverage, or the PR checklist. For a generated waiver of one skipped test, the omitted sections carry little weight, so the description is mostly complete.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

[tensorrt-cicd]

PR_Github #80135 [ skip ] triggered by Bot. Commit: f18f0aa Link to invocation

[tensorrt-cicd]

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

Link to invocation