[TRTLLM-8638][fix] Add failed cases into waives.txt

[xinhe-nv]

waive failed cases.

Summary by CodeRabbit

• Tests
  • Added test entries to the waives list for improved test management.

[xinhe-nv]

/bot run --skip-test

[coderabbitai]

📝 Walkthrough

Walkthrough

Two new test cases are added to the waives list and marked as SKIP: test_ptp_quickstart_multimodal (with Mistral Small parameters) and test_trtllm_multimodal_benchmark_serving. No functional changes.

Changes

Cohort / File(s)	Summary
Test Waives Configuration tests/integration/test_lists/waives.txt	Added two new SKIP entries: test_e2e.py::test_ptp_quickstart_multimodal[mistral-small-3.1-24b-instruct-Mistral-Small-3.1-24B-Instruct-2503-mixture_text_image-True] and test_e2e.py::test_trtllm_multimodal_benchmark_serving

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• [None][chore] Add failed cases into waives.txt #8872: Modifies the same waives list to add SKIP/waive entries for failing tests including multimodal/quickstart cases
• [None][chore] Add failed cases into waives.txt #7735: Also updates tests/integration/test_lists/waives.txt to add SKIP entries for failing tests
• [None][chore] Add failed cases into waives.txt #8998: Modifies the same waives file to add new SKIP/waived test entries

Suggested reviewers

• crazydemo
• LarryXFly
• kaiyux

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description is extremely minimal and lacks the required information sections specified in the template (Description, Test Coverage, and PR Checklist).	Expand the description to include: explanation of which tests are being waived and why they failed; relevant test coverage information; and completion of the PR checklist items.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title accurately describes the main change: adding failed test cases to the waives.txt file.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

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

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[tensorrt-cicd]

PR_Github #24532 [ run ] triggered by Bot. Commit: f11bf6b

[xinhe-nv]

/bot reuse-pipeline

[tensorrt-cicd]

PR_Github #24563 [ reuse-pipeline ] triggered by Bot. Commit: eceea5a

[tensorrt-cicd]

PR_Github #24532 [ run ] completed with state ABORTED. Commit: f11bf6b
LLM/main/L0_MergeRequest_PR #18517 (Blue Ocean) completed with status: ABORTED

[tensorrt-cicd]

PR_Github #24563 [ reuse-pipeline ] completed with state SUCCESS. Commit: eceea5a
Can't reuse PR_Github #24532 (Partly Tested) with status: ABORTED

[xinhe-nv]

/bot run --skip-test

[tensorrt-cicd]

PR_Github #24564 [ run ] triggered by Bot. Commit: eceea5a

[tensorrt-cicd]

PR_Github #24564 [ run ] completed with state SUCCESS. Commit: eceea5a
/LLM/main/L0_MergeRequest_PR pipeline #18541 (Partly Tested) completed with status: 'SUCCESS'

[xinhe-nv]

/bot reuse-pipeline

[tensorrt-cicd]

PR_Github #24597 [ reuse-pipeline ] triggered by Bot. Commit: d34b7f5

[tensorrt-cicd]

PR_Github #24597 [ reuse-pipeline ] completed with state SUCCESS. Commit: d34b7f5
Reusing PR_Github #24564 (Partly Tested) for commit d34b7f5