Skip to content

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

Merged
jieli-matrix merged 5 commits into
NVIDIA:mainfrom
xinhe-nv:trtllm-ci-report/waive-20260422-100521
Apr 23, 2026
Merged

[None][infra] Waive 2 failed cases for main in post-merge#13335
jieli-matrix merged 5 commits into
NVIDIA:mainfrom
xinhe-nv:trtllm-ci-report/waive-20260422-100521

Conversation

@xinhe-nv

@xinhe-nv xinhe-nv commented Apr 22, 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): 6026676, 6069543

Waive entries added

accuracy/test_llm_api_pytorch.py::TestGPTOSS::test_w4_chunked_prefill[triton-auto] SKIP (https://nvbugs/6026676)
accuracy/test_llm_api_pytorch.py::TestQwen3_5_35B_A3B::test_bf16 SKIP (https://nvbugs/6069543)

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

Summary by CodeRabbit

  • Tests
    • Added two test cases to the skip list to prevent execution during automated test runs.

Bug(s): 6026676, 6069543
Requested by: qa@nvidia.com

Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
@jieli-matrix
jieli-matrix enabled auto-merge (squash) April 22, 2026 10:06
@jieli-matrix

Copy link
Copy Markdown
Collaborator

/bot run --stage-list ""

@coderabbitai

coderabbitai Bot commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 88b0ec8c-fdf9-4042-9958-b07b2077e3dc

📥 Commits

Reviewing files that changed from the base of the PR and between 36fb5f0 and 8262587.

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

📝 Walkthrough

Walkthrough

This PR adds two new test case skip entries to the waive list, referencing specific GPU bug issues (nvbugs/6026676 and nvbugs/6069543) for LLM API PyTorch tests.

Changes

Cohort / File(s) Summary
Test Waive List
tests/integration/test_lists/waives.txt
Added two skipped test entries: accuracy/test_llm_api_pytorch.py::TestGPTOSS::test_w4_chunked_prefill[triton-auto] and accuracy/test_llm_api_pytorch.py::TestQwen3_5_35B_A3B::test_bf16 with associated NVIDIA bug references.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • jieli-matrix
  • ZhanruiSunCh
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description is largely incomplete compared to the template; it lacks formal Description, Test Coverage, and PR Checklist sections, though it provides essential information about the auto-generated waive entries and bug references. Consider adding a formal Description section explaining why the waives are necessary, and confirm Test Coverage and PR Checklist sections even if auto-generated PRs follow different conventions.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: waiving 2 failed test cases for the main branch in post-merge, with appropriate ticket format and type label.
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.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #44956 [ run ] triggered by Bot. Commit: 8262587 Link to invocation

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

Copy link
Copy Markdown
Collaborator

/bot run --stage-list ""

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #45065 [ run ] triggered by Bot. Commit: 7fa74c0 Link to invocation

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

Copy link
Copy Markdown
Collaborator

PR_Github #45065 [ run ] completed with state SUCCESS. Commit: 7fa74c0
/LLM/main/L0_MergeRequest_PR pipeline #35368 (Partly Tested) completed with status: 'SUCCESS'

CI Report

Link to invocation

@xinhe-nv

Copy link
Copy Markdown
Collaborator Author

/bot reuse-pipeline

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #45146 [ reuse-pipeline ] triggered by Bot. Commit: 555ec0a Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #45146 [ reuse-pipeline ] completed with state SUCCESS. Commit: 555ec0a
Reusing PR_Github #45065 (Partly Tested) for commit 555ec0a

Link to invocation

@jieli-matrix
jieli-matrix merged commit cf20457 into NVIDIA:main Apr 23, 2026
5 checks passed
@xinhe-nv
xinhe-nv deleted the trtllm-ci-report/waive-20260422-100521 branch April 23, 2026 07:59
ziyixiong-nv pushed a commit to ziyixiong-nv/TensorRT-LLM that referenced this pull request Apr 24, 2026
Signed-off-by: xinhe-nv <200704525+xinhe-nv@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