[None][chore] Add failed cases into waives.txt by xinhe-nv · Pull Request #11392 · NVIDIA/TensorRT-LLM

xinhe-nv · 2026-02-09T18:36:55Z

waive failed cases.

Summary by CodeRabbit

Tests
- Updated test configuration to skip a specific test case due to a known issue.

Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>

Signed-off-by: Jie Li <76780849+jieli-matrix@users.noreply.github.com>

jieli-matrix · 2026-02-10T07:25:04Z

/bot run --stage-list ""

coderabbitai · 2026-02-10T07:26:52Z

📝 Walkthrough

Walkthrough

This PR adds a single test skip entry to the waives.txt file for test_openai_chat_guided_decoding[openai/gpt-oss-120b], referencing bug nvbugs/5884677. No behavioral or control-flow changes.

Changes

Cohort / File(s)	Summary
Test Waiver `tests/integration/test_lists/waives.txt`	Added skip entry for `test_e2e.py::test_openai_chat_guided_decoding[openai/gpt-oss-120b]` with bug reference.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

[None][chore] Add failed cases into waives.txt #11193: Modifies the same waives.txt file by adding SKIP entries for failing tests.
[TRTLLM-8638][fix] Add failed cases into waives.txt #10132: Modifies tests/integration/test_lists/waives.txt to add SKIP entries for failing tests.
[None][chore] Add failed cases into waives.txt #10321: Modifies the same waives.txt file by adding SKIP/waived test entries.

Suggested reviewers

StanleySun639
LarryXFly
crazydemo

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description is minimal and lacks required sections from the template (Description, Test Coverage, PR Checklist). While it mentions the purpose, it does not follow the repository's description template structure.	Expand the description to include the required template sections: a brief explanation of what failed cases are being waived and why, test coverage information, and completion of the PR checklist items.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and accurately describes the main change: adding failed test cases to the waives.txt file as a chore task.
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

No actionable comments were generated in the recent review. 🎉

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

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

tensorrt-cicd · 2026-02-10T07:30:55Z

PR_Github #35453 [ run ] triggered by Bot. Commit: 49f68ae

tensorrt-cicd · 2026-02-10T08:17:09Z

PR_Github #35453 [ run ] completed with state SUCCESS. Commit: 49f68ae
/LLM/main/L0_MergeRequest_PR pipeline #27383 (Partly Tested) completed with status: 'SUCCESS'

…ION_TEST_1924 Signed-off-by: Jie Li <76780849+jieli-matrix@users.noreply.github.com>

xinhe-nv · 2026-02-10T09:37:54Z

/bot run --skip-test

xinhe-nv · 2026-02-10T09:42:43Z

/bot reuse-pipeline

tensorrt-cicd · 2026-02-10T09:44:14Z

PR_Github #35483 [ run ] triggered by Bot. Commit: f14ad87

tensorrt-cicd · 2026-02-10T09:48:12Z

PR_Github #35487 [ reuse-pipeline ] triggered by Bot. Commit: f14ad87

tensorrt-cicd · 2026-02-10T09:48:17Z

PR_Github #35483 [ run ] completed with state ABORTED. Commit: f14ad87

…ION_TEST_1924

xinhe-nv · 2026-02-10T09:55:11Z

/bot reuse-pipeline

…ION_TEST_1924

xinhe-nv · 2026-02-10T09:55:55Z

/bot reuse-pipeline

tensorrt-cicd · 2026-02-10T10:01:12Z

PR_Github #35490 [ reuse-pipeline ] triggered by Bot. Commit: a0bd514

tensorrt-cicd · 2026-02-10T10:01:15Z

PR_Github #35487 [ reuse-pipeline ] completed with state ABORTED. Commit: f14ad87
Can't reuse PR_Github #0 with status: UNKNOWN

tensorrt-cicd · 2026-02-10T10:01:38Z

PR_Github #35491 [ reuse-pipeline ] triggered by Bot. Commit: a0bd514

tensorrt-cicd · 2026-02-10T10:01:40Z

PR_Github #35490 [ reuse-pipeline ] completed with state ABORTED. Commit: a0bd514
Can't reuse PR_Github #0 with status: UNKNOWN

…ION_TEST_1924 Signed-off-by: Jie Li <76780849+jieli-matrix@users.noreply.github.com>

jieli-matrix · 2026-02-10T10:51:21Z

/bot reuse-pipeline

tensorrt-cicd · 2026-02-10T10:57:32Z

PR_Github #35499 [ reuse-pipeline ] triggered by Bot. Commit: b84898a

tensorrt-cicd · 2026-02-10T10:57:48Z

PR_Github #35491 [ reuse-pipeline ] completed with state ABORTED. Commit: a0bd514
Can't reuse PR_Github #35453 (Partly Tested) with status: SUCCESS

jieli-matrix · 2026-02-10T11:09:18Z

/bot run --stage-list ""

tensorrt-cicd · 2026-02-10T11:15:04Z

PR_Github #35500 [ run ] triggered by Bot. Commit: b84898a

tensorrt-cicd · 2026-02-10T11:15:20Z

PR_Github #35499 [ reuse-pipeline ] completed with state ABORTED. Commit: b84898a
Can't reuse PR_Github #0 with status: UNKNOWN

jieli-matrix · 2026-02-10T11:17:13Z

/bot run --stage-list ""

tensorrt-cicd · 2026-02-10T11:23:25Z

PR_Github #35502 [ run ] triggered by Bot. Commit: b84898a

tensorrt-cicd · 2026-02-10T11:23:42Z

PR_Github #35500 [ run ] completed with state ABORTED. Commit: b84898a

tensorrt-cicd · 2026-02-10T13:12:18Z

PR_Github #35502 [ run ] completed with state SUCCESS. Commit: b84898a
/LLM/main/L0_MergeRequest_PR pipeline #27413 (Partly Tested) completed with status: 'SUCCESS'

…ION_TEST_1924

xinhe-nv · 2026-02-11T03:42:03Z

/bot reuse-pipeline

…ION_TEST_1924

xinhe-nv · 2026-02-11T10:57:02Z

/bot reuse-pipeline

…ION_TEST_1924

xinhe-nv · 2026-02-12T01:51:06Z

/bot run --stage-list ""

…ION_TEST_1924

EmmaQiaoCh · 2026-02-12T01:57:53Z

/bot skip --comment "Waive failed case"

xinhe-nv · 2026-02-12T01:57:58Z

/bot skip --comment “check list is passed”

tensorrt-cicd · 2026-02-12T02:04:57Z

PR_Github #35692 Bot args parsing error: usage: /bot [-h]
{run,kill,skip,submit,reviewers,reuse-pipeline,reuse-review} ...
/bot: error: unrecognized arguments: list is passed”

tensorrt-cicd · 2026-02-12T02:05:39Z

PR_Github #35693 [ skip ] triggered by Bot. Commit: f8b49ba

tensorrt-cicd · 2026-02-12T02:52:28Z

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

Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com> Signed-off-by: Jie Li <76780849+jieli-matrix@users.noreply.github.com> Co-authored-by: Jie Li <76780849+jieli-matrix@users.noreply.github.com>

xinhe-nv requested review from LarryXFly, StanleySun639, crazydemo and jieli-matrix February 9, 2026 18:36

xinhe-nv and others added 2 commits February 10, 2026 15:24

update waive list

bca4142

Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>

Update waives.txt

49f68ae

Signed-off-by: Jie Li <76780849+jieli-matrix@users.noreply.github.com>

jieli-matrix force-pushed the user/qa/post_update_waive_20260210_LLM_FUNCTION_TEST_1924 branch from a4cddc6 to 49f68ae Compare February 10, 2026 07:24

jieli-matrix marked this pull request as ready for review February 10, 2026 07:24

jieli-matrix enabled auto-merge (squash) February 10, 2026 07:24

crazydemo approved these changes Feb 10, 2026

View reviewed changes

StanleySun639 approved these changes Feb 10, 2026

View reviewed changes

Merge branch 'main' into user/qa/post_update_waive_20260210_LLM_FUNCT…

f14ad87

…ION_TEST_1924 Signed-off-by: Jie Li <76780849+jieli-matrix@users.noreply.github.com>

Merge branch 'main' into user/qa/post_update_waive_20260210_LLM_FUNCT…

e19e1dd

…ION_TEST_1924

Merge branch 'main' into user/qa/post_update_waive_20260210_LLM_FUNCT…

a0bd514

…ION_TEST_1924

Merge branch 'main' into user/qa/post_update_waive_20260210_LLM_FUNCT…

b84898a

…ION_TEST_1924 Signed-off-by: Jie Li <76780849+jieli-matrix@users.noreply.github.com>

xinhe-nv added 2 commits February 11, 2026 09:31

Merge branch 'main' into user/qa/post_update_waive_20260210_LLM_FUNCT…

21117f8

…ION_TEST_1924

Merge branch 'main' into user/qa/post_update_waive_20260210_LLM_FUNCT…

58a3d29

…ION_TEST_1924

xinhe-nv added 3 commits February 11, 2026 13:57

Merge branch 'main' into user/qa/post_update_waive_20260210_LLM_FUNCT…

da04568

…ION_TEST_1924

Merge branch 'main' into user/qa/post_update_waive_20260210_LLM_FUNCT…

bbab229

…ION_TEST_1924

Merge branch 'main' into user/qa/post_update_waive_20260210_LLM_FUNCT…

1f75f79

…ION_TEST_1924

Merge branch 'main' into user/qa/post_update_waive_20260210_LLM_FUNCT…

6c31549

…ION_TEST_1924

Merge branch 'main' into user/qa/post_update_waive_20260210_LLM_FUNCT…

f8b49ba

…ION_TEST_1924

jieli-matrix merged commit 4264873 into NVIDIA:main Feb 12, 2026
5 checks passed

xinhe-nv deleted the user/qa/post_update_waive_20260210_LLM_FUNCTION_TEST_1924 branch February 12, 2026 03:09

Uh oh!

Conversation

xinhe-nv commented Feb 9, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

jieli-matrix commented Feb 10, 2026

Uh oh!

coderabbitai Bot commented Feb 10, 2026

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

Uh oh!

tensorrt-cicd commented Feb 10, 2026

Uh oh!

tensorrt-cicd commented Feb 10, 2026

Uh oh!

xinhe-nv commented Feb 10, 2026

Uh oh!

xinhe-nv commented Feb 10, 2026

Uh oh!

tensorrt-cicd commented Feb 10, 2026

Uh oh!

tensorrt-cicd commented Feb 10, 2026

Uh oh!

tensorrt-cicd commented Feb 10, 2026

Uh oh!

xinhe-nv commented Feb 10, 2026

Uh oh!

xinhe-nv commented Feb 10, 2026

Uh oh!

tensorrt-cicd commented Feb 10, 2026

Uh oh!

tensorrt-cicd commented Feb 10, 2026

Uh oh!

tensorrt-cicd commented Feb 10, 2026

Uh oh!

tensorrt-cicd commented Feb 10, 2026

Uh oh!

jieli-matrix commented Feb 10, 2026

Uh oh!

tensorrt-cicd commented Feb 10, 2026

Uh oh!

tensorrt-cicd commented Feb 10, 2026

Uh oh!

jieli-matrix commented Feb 10, 2026

Uh oh!

tensorrt-cicd commented Feb 10, 2026

Uh oh!

tensorrt-cicd commented Feb 10, 2026

Uh oh!

jieli-matrix commented Feb 10, 2026

Uh oh!

tensorrt-cicd commented Feb 10, 2026

Uh oh!

tensorrt-cicd commented Feb 10, 2026

Uh oh!

tensorrt-cicd commented Feb 10, 2026

Uh oh!

xinhe-nv commented Feb 11, 2026

Uh oh!

xinhe-nv commented Feb 11, 2026

Uh oh!

xinhe-nv commented Feb 12, 2026

Uh oh!

EmmaQiaoCh commented Feb 12, 2026

Uh oh!

xinhe-nv commented Feb 12, 2026

Uh oh!

tensorrt-cicd commented Feb 12, 2026

Uh oh!

tensorrt-cicd commented Feb 12, 2026

Uh oh!

tensorrt-cicd commented Feb 12, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

xinhe-nv commented Feb 9, 2026 •

edited by coderabbitai Bot

Loading