Skip to content

[None][test] Waive 4 failed cases for main in QA CI#16032

Merged
jieli-matrix merged 3 commits into
NVIDIA:mainfrom
trtllm-agent:trtllm-ci-report/waive-20260707-034810
Jul 7, 2026
Merged

[None][test] Waive 4 failed cases for main in QA CI#16032
jieli-matrix merged 3 commits into
NVIDIA:mainfrom
trtllm-agent:trtllm-ci-report/waive-20260707-034810

Conversation

@trtllm-agent

@trtllm-agent trtllm-agent commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by qa@nvidia.com)
Target branch: main
Bug(s): 6414760, 6415323, 6422318

Waive entries added

full:GB200/disaggregated/test_ad_disagg.py::test_async_generation_matches_aggregate SKIP (https://nvbugs/6415323)
full:GB200/accuracy/test_llm_api_pytorch_multimodal.py::TestExaone4_5_33B::test_auto_dtype[forced_chunked_prefill] SKIP (https://nvbugs/6422318)
full:GB200/accuracy/test_llm_api_pytorch_multimodal.py::TestExaone4_5_33B::test_auto_dtype[full_budget] SKIP (https://nvbugs/6422318)
full:GB200/test_e2e.py::test_qwen_e2e_cpprunner_large_new_tokens[DeepSeek-R1-Distill-Qwen-1.5B-DeepSeek-R1-Distill-Qwen-1.5B] SKIP (https://nvbugs/6414760)

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

Summary by CodeRabbit

  • Chores
    • Updated internal test exclusions to better reflect current GB200 multimodal and disaggregated-serving coverage.
    • Added a few additional test cases to the waiver list to keep automated validation stable.

Bug(s): 6414760, 6415323, 6422318
Requested by: qa@nvidia.com

Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
@jieli-matrix

Copy link
Copy Markdown
Collaborator

/bot skip --comment "Only waive tests"

@coderabbitai

coderabbitai Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

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: 174ed6c4-342c-4fc3-9320-3d61493d86d4

📥 Commits

Reviewing files that changed from the base of the PR and between 790ea15 and 6788a3f.

📒 Files selected for processing (1)
  • tests/integration/test_lists/waives.txt

📝 Walkthrough

Walkthrough

Added four new SKIP waiver entries to tests/integration/test_lists/waives.txt covering GB200 multimodal (TestExaone4_5_33B.test_auto_dtype), disaggregated (test_async_generation_matches_aggregate), and end-to-end (test_qwen_e2e_cpprunner_large_new_tokens) test cases.

Changes

Waiver List Updates

Layer / File(s) Summary
Add new GB200 SKIP entries
tests/integration/test_lists/waives.txt
Adds two TestExaone4_5_33B.test_auto_dtype SKIP entries, a test_async_generation_matches_aggregate SKIP entry, and a test_qwen_e2e_cpprunner_large_new_tokens SKIP entry to the GB200 waiver block.

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

Possibly related PRs

Suggested reviewers: crazydemo, LarryXFly, StanleySun639, xinhe-nv, jieli-matrix

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description explains the waiver entries, but it misses the required template sections for Description, Test Coverage, and PR Checklist. Add the required template sections with a short issue summary, test coverage, and the PR checklist confirmation.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and accurately summarizes the main change: waiving four QA CI failures.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #57924 [ skip ] triggered by Bot. Commit: 6788a3f Link to invocation

@jieli-matrix jieli-matrix disabled auto-merge July 7, 2026 03:57
@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #57924 [ skip ] completed with state SUCCESS. Commit: 6788a3f
Skipping testing for commit 6788a3f

Link to invocation

Signed-off-by: Jie Li <76780849+jieli-matrix@users.noreply.github.com>
@jieli-matrix jieli-matrix enabled auto-merge (squash) July 7, 2026 05:03
@jieli-matrix

Copy link
Copy Markdown
Collaborator

/bot reuse-pipeline

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #57938 [ reuse-pipeline ] triggered by Bot. Commit: e9d2f57 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #57938 [ reuse-pipeline ] completed with state SUCCESS. Commit: e9d2f57
Can't reuse PR_Github #0 with status: UNKNOWN

Link to invocation

@jieli-matrix

Copy link
Copy Markdown
Collaborator

/bot skip --comment "Only waive tests"

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #57980 [ skip ] triggered by Bot. Commit: c5f254b Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

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

Link to invocation

@jieli-matrix jieli-matrix merged commit 32a953b into NVIDIA:main Jul 7, 2026
7 checks passed
BrianLi23 pushed a commit to BrianLi23/TensorRT-LLM that referenced this pull request Jul 9, 2026
Signed-off-by: trtllm-agent <296075020+trtllm-agent@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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants