[None][infra] Waive 2 failed cases for main in post-merge 2584

[ZhanruiSunCh]

Auto-generated Waive PR

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

Waive entries added

stress_test/stress_test.py::test_run_stress_test[llama-v3-8b-instruct-hf_tp1-stress_time_300s_timeout_450s-MAX_UTILIZATION-pytorch-stress-test] SKIP (https://nvbugs/5955927)
stress_test/stress_test.py::test_run_stress_test[llama-v3-8b-instruct-hf_tp1-stress_time_300s_timeout_450s-GUARANTEED_NO_EVICT-pytorch-stress-test] SKIP (https://nvbugs/5955927)

---

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

Summary by CodeRabbit

Release Notes

• Tests
  • Updated test entry reference for improved tracking
  • Added two new stress test scenarios to expand performance validation coverage under sustained load conditions

[EmmaQiaoCh]

/bot skip --comment "Waive failed cases"

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d5644de2-1b1c-4207-b3e8-caf49b504d32

📥 Commits

Reviewing files that changed from the base of the PR and between 298b6c8 and f79df88.

📒 Files selected for processing (1)

• tests/integration/test_lists/waives.txt

---

📝 Walkthrough

Walkthrough

Updates test waives configuration by changing a perf test bug reference from nvbugs/4846166 to nvbugs/5846166 and adds two new stress test waive entries for llama-v3-8b-instruct-hf test cases.

Changes

Cohort / File(s)	Summary
Test Waives Configuration tests/integration/test_lists/waives.txt	Updated perf test bug reference URL for disagg_upload test case from nvbugs/4846166 to nvbugs/5846166; added two new waive entries for stress_test llama-v3-8b-instruct-hf test cases with different memory utilization profiles.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• [None][infra] Waive 5 failed cases for main in post-merge 2578 #12023: Modifies the same waives.txt file and updates the same perf test NVBugs URL reference for disagg_upload case.
• [None][infra] Waive 2 failed cases for release/1.2 in post-merge 42 #11971: Adds the identical two stress_test waive entries for llama-v3-8b-instruct-hf test cases.
• [None][infra] Waive failed cases for main on 1/30 #11142: Adjusts waived test entries in the same waives configuration file.

Suggested reviewers

• chzblych

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description deviates significantly from the required template structure; it lacks key sections like 'Description', 'Test Coverage', and the PR Checklist, presenting instead an auto-generated format.	Restructure the description to follow the required template: add explicit 'Description' and 'Test Coverage' sections explaining the waived tests and their purpose, and include the PR Checklist with appropriate items checked.

✅ Passed checks (2 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, with a reference to the post-merge run number.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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

[tensorrt-cicd]

PR_Github #38572 [ skip ] triggered by Bot. Commit: f79df88 Link to invocation

[tensorrt-cicd]

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

Link to invocation