[https://nvbugs/6435097][fix] Remove the single stale nvbugs/6427411 waiver line for…

[trtllm-agent]

Summary

• Root cause: PR [None][perf] Move greedy stop checks to host #15920 added a use_host_stop_criteria fast path that under-populated per-step/seq_slot/beam token buffers on non-last PP ranks, causing IndexError; PR [TRTLLM-14155][fix] Revert host-side greedy stop checks from #15920 #16163 (commit 9a8ec05) reverted PR [None][perf] Move greedy stop checks to host #15920 and is present at HEAD, so the code is already fixed. The waiver line for this test was left behind by cleanup PRs [https://nvbugs/6427411][chore] Waive failed tests in Pre-merge #16103/[https://nvbugs/6427411][chore] Waive a failed test in Pre-merge #16105/[https://nvbugs/6427411][chore] Waive remaining PP sampler regressions #16127.
• Fix: Remove the single stale nvbugs/6427411 waiver line for test_ctx_pp_gen_tp_asymmetric[GSM8K-gen_tp=1-ctx_pp=2] in waives.txt; do not touch other test entries or other bug IDs.
• Automated fix generated by repair-bot

Test plan

• Verify fix on the same GPU type as the original failure
• Check for regressions in related tests

Links

• Bug: https://nvbugs/6435097

Dev Engineer Review

• Removed the stale nvbugs/6427411 waiver and SKIP marker for TestLlama3_1_8BInstruct::test_ctx_pp_gen_tp_asymmetric[GSM8K-gen_tp=1-ctx_pp=2].
• The change is limited to one waives.txt entry.
• The test-list format remains consistent.
• No other tests, bug references, or public entities changed.

QA Engineer Review

• No test-db/ or qa/ files were modified.
• Removed one entry from tests/integration/test_lists/waives.txt.
• The affected test will run without the obsolete waiver.
• Verdict: needs follow-up because CBTS coverage data is unavailable.

[coderabbitai]

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: f03ee1b4-3c6f-4391-97ad-521fdd0091f6

📥 Commits

Reviewing files that changed from the base of the PR and between 8e588da and f0380c6.

📒 Files selected for processing (1)

• tests/integration/test_lists/waives.txt

💤 Files with no reviewable changes (1)

• tests/integration/test_lists/waives.txt

---

Walkthrough

The integration waiver list removes the SKIP entry for the Llama 3.1 8B Instruct asymmetric context/parallelism test configuration.

Changes

Cohort / File(s)	Summary
Waiver list update tests/integration/test_lists/waives.txt	Removes the waiver and nvbugs reference for test_ctx_pp_gen_tp_asymmetric with GSM8K-gen_tp=1-ctx_pp=2.

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

Possibly related PRs

• NVIDIA/TensorRT-LLM#15585: Modifies the same integration waiver list.
• NVIDIA/TensorRT-LLM#17219: Removes an obsolete waiver entry from the same list.
• NVIDIA/TensorRT-LLM#17309: Removes a specific skipped-test waiver from the same list.

Suggested labels: ci: full pre-merge approved

Suggested reviewers: crazydemo, brnguyen2, mlefeb01

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description check	✅ Passed	The description explains the root cause, the stale waiver removal, scope limits, test plan, and bug reference.
Title check	✅ Passed	The title identifies the bug, fix type, and removal of the stale waiver, but it ends with an ellipsis and an incomplete test reference.
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.}

[BowenFu]

LGTM — removes a stale skip; the under-populated-buffer regression it guarded was already reverted (#16163). No product code.

[mzweilz]

LGTM if the unwaived test passes.

[fredricz-20070104]

Review summary - Approve

Reviewed the full diff; no blocking or major issues found.

Minor, non-blocking notes:

• tests/integration/test_lists/waives.txt: Sibling waiver for the same bug id 6427411 left in place

Automated review by NVCortex Lite, run by @fredricz-20070104.

[coderabbitai]

Note

GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer.