[None][infra] Remove stale perf-sanity waives orphaned by #17609

[brnguyen2]

Description

PR #17609 de-enrolled a batch of perf/test_perf_sanity.py::test_e2e DeepSeek-V3.2 / Kimi-K2.5 cases from the test-db perf_sanity lists but left their entries in tests/integration/test_lists/waives.txt. Those 18 entries now reference tests that exist in no L0 or QA list, so the Check Test List stage (scripts/check_test_list.py --l0 --qa --waive) fails with:

Non-existent test name in l0 or qa list found in waives.txt: perf/test_perf_sanity.py::test_e2e[...]

This change removes the 18 orphaned entries. Their nvbug SKIPs are moot now that the referenced tests no longer run.

Why this escaped CI

Merge skew between two independently-green PRs:

• [None][infra] Waive 20 failed cases for main in post-merge 2899 #17593 (auto-waiver) added the perf-sanity waives while the tests still existed, so its Check Test List passed.
• [None][test] Nemotron-Ultra-V3 perf-sanity cases (GB300); de-enroll DeepSeek-V3.2, Kimi-K2.5 & Llama cases #17609 later removed those test cases from the lists but did not touch waives.txt. Its gating pipeline validated a tree that did not contain both the list removals and the pre-existing waives together, so nothing checked them as a unit before merge.

The result only surfaces on a subsequent PR that runs Check Test List fresh (no stage reuse), which is why it was not caught at #17609 merge time and why unrelated PRs get flagged for it.

Test Coverage

No new code paths. Validated locally:

• scripts/check_test_list.py --validate (AST list validation) passes.
• scripts/check_test_list.py --check-duplicate-waives reports no duplicates.
• Confirmed no remaining perf_sanity waive references a test absent from the active L0/QA lists.

PR Checklist

• Please check this after reviewing the above items as appropriate for this PR.

[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: f50218b2-1ef7-48c2-8e9c-e1caa640be6e

📥 Commits

Reviewing files that changed from the base of the PR and between a098858 and c42fdcd.

📒 Files selected for processing (1)

• tests/integration/test_lists/waives.txt

💤 Files with no reviewable changes (1)

• tests/integration/test_lists/waives.txt

Included review availability: Your plan includes up to 12 reviews per rolling hour; 11 remain after this review.

---

Walkthrough

The integration waiver list removes obsolete performance skips and adds updated GB300 skips for DeepSeek V4 Pro, GLM-5, DeepSeek-R1, and GPT-OSS configurations.

Changes

Integration waiver maintenance

Layer / File(s)	Summary
Performance waiver entries tests/integration/test_lists/waives.txt	Removed obsolete DeepSeek V3.2, GB200, and Kimi K2.5 skips. Added updated GB300 aggregate and disaggregated skips for DeepSeek V4 Pro, GLM-5, DeepSeek-R1, and GPT-OSS.

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

Merge Risk: ⚪ Minimal · up to c42fd

This localized cleanup removes obsolete test waivers without changing runtime code or behavior, and no actionable merge-blocking risk remains after normal checks and review.

Possibly related PRs

• NVIDIA/TensorRT-LLM#17782: Updates overlapping GLM-5 and GLM52 integration waiver entries.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
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.
Title check	✅ Passed	The title clearly identifies the infrastructure cleanup and removal of stale performance-test waivers orphaned by issue #17609.
Description check	✅ Passed	The description explains the issue, root cause, solution, validation steps, and checklist status with sufficient detail for this cleanup change.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

[brnguyen2]

/bot run

[tensorrt-cicd]

PR_Github #66577 [ run ] triggered by Bot. Commit: c42fdcd Link to invocation

[tensorrt-cicd]

PR_Github #66577 [ run ] completed with state FAILURE. Commit: c42fdcd
/LLM/main/L0_MergeRequest_PR pipeline #54207 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

• Please check the failed tests and fix your PR
• If you cannot view the failures, ask the CI triggerer to share details
• Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

[brnguyen2]

/bot skip --comment "waive only PR"

[tensorrt-cicd]

PR_Github #66588 [ skip ] triggered by Bot. Commit: c42fdcd Link to invocation

[tensorrt-cicd]

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

Link to invocation