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

[xinhe-nv]

waive failed cases.

Summary by CodeRabbit

• Tests
  • Updated test skip list configuration.

Note: This release contains no user-visible changes. The update addresses internal test infrastructure only.

[xinhe-nv]

/bot run --skip-test

[coderabbitai]

📝 Walkthrough

Walkthrough

Added a single SKIP entry to the test waives list for a test_ptp_quickstart_multimodal variant with Qwen2.5-VL-7B model, referencing an associated bug ticket. No executable logic changes; test configuration only.

Changes

Cohort / File(s)	Change Summary
Test waivers tests/integration/test_lists/waives.txt	Added SKIP entry for test_e2e.py::test_ptp_quickstart_multimodal[qwen2.5-vl-7b-instruct-Qwen2-VL-7B-Instruct-image-False] with bug reference

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• PR [TRTLLM-8638][fix] Add failed cases into waives.txt #8486: Adds SKIP entries for other ptp_quickstart_multimodal end-to-end test variants to the same waives list.
• PR [https://nvbugs/5444095][infra] waive test_ptp_quickstart_multimodal llava test #6795: Adds SKIP waivers for the same test_ptp_quickstart_multimodal test function with different model variants.
• PR [None][chore] Add failed cases into waives.txt #7986: Modifies waives.txt to add multiple SKIP entries for failing tests in the same test-skipping configuration file.

Suggested reviewers

• crazydemo
• LarryXFly

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description Check	⚠️ Warning	The PR description is largely incomplete compared to the repository's template. The provided description consists of only one brief sentence ("waive failed cases.") and lacks the required sections including an explanation of the issue and solution in the Description section, Test Coverage information, and the PR Checklist items. While the description is not off-topic, it fails to provide substantive context about the change, why it was necessary, or which tests validate the modification.	The author should expand the PR description to include the missing template sections. At minimum, please add a Description section explaining which test cases were failing and why they are being waived, a Test Coverage section identifying relevant tests, and ensure the PR Checklist items are addressed. This will provide reviewers with sufficient context to understand the change's purpose and impact.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.
Title Check	✅ Passed	The pull request title "[TRTLLM-8638][fix] Add flaky failed cases into waives.txt" accurately describes the main change in the changeset. The PR adds a new test entry to the waives.txt file, which is a test-skipping list, exactly as the title indicates. The title is specific about the action (Add), the destination (waives.txt), and the context (fix for TRTLLM-8638), making it clear to anyone reviewing the git history. The title is concise, avoids vague terms, and directly reflects the primary change in the pull request.

✨ 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 #22262 [ run ] triggered by Bot. Commit: d75056a

[tensorrt-cicd]

PR_Github #22262 [ run ] completed with state SUCCESS. Commit: d75056a
/LLM/main/L0_MergeRequest_PR pipeline #16784 (Partly Tested) completed with status: 'FAILURE'

[xinhe-nv]

/bot run --skip-test

[tensorrt-cicd]

PR_Github #22302 [ run ] triggered by Bot. Commit: 8468c24

[tensorrt-cicd]

PR_Github #22302 [ run ] completed with state SUCCESS. Commit: 8468c24
/LLM/main/L0_MergeRequest_PR pipeline #16813 (Partly Tested) completed with status: 'SUCCESS'