Skip to content

[None][test] Waive 1 failed cases for main in QA CI - #16062

Closed
trtllm-agent wants to merge 3 commits into
NVIDIA:mainfrom
trtllm-agent:trtllm-ci-report/waive-20260707-094808
Closed

[None][test] Waive 1 failed cases for main in QA CI#16062
trtllm-agent wants to merge 3 commits into
NVIDIA:mainfrom
trtllm-agent:trtllm-ci-report/waive-20260707-094808

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): 6423991

Waive entries added

full:H20/accuracy/test_llm_api_pytorch.py::TestQwen3_5_35B_A3B::test_fp8_moe_dflash SKIP (https://nvbugs/6423991)

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

Summary by CodeRabbit

  • Tests
    • Added a new test waiver entry to skip a specific integration test case due to a known issue.

Bug(s): 6423991
Requested by: qa@nvidia.com

Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
@jieli-matrix
jieli-matrix enabled auto-merge (squash) July 7, 2026 09:48
@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: 061a3c13-2317-46e4-83cd-e38f0dc1268c

📥 Commits

Reviewing files that changed from the base of the PR and between ce67288 and b337ce0.

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

📝 Walkthrough

Walkthrough

This PR adds a single new SKIP waiver entry to tests/integration/test_lists/waives.txt for the H20 accuracy test test_llm_api_pytorch.py::TestQwen3_5_35B_A3B::test_fp8_moe_dflash, referencing an associated nvbugs link.

Changes

Test waiver update

Layer / File(s) Summary
Add SKIP waiver entry
tests/integration/test_lists/waives.txt
Added a new SKIP waiver line for the H20 TestQwen3_5_35B_A3B::test_fp8_moe_dflash test case with an nvbugs reference.

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

Possibly related PRs

  • NVIDIA/TensorRT-LLM#16030: Both PRs modify waives.txt to add/adjust H20-related SKIP entries for test_llm_api_pytorch, including Qwen tests.
  • NVIDIA/TensorRT-LLM#14136: Both PRs add SKIP waiver entries for failing accuracy/test_llm_api_pytorch.py test_fp8 cases.
  • NVIDIA/TensorRT-LLM#13335: Both PRs add test_fp8_moe_dflash/related test-skip entries for test_llm_api_pytorch.py under Qwen3.5.

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

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description omits the required Description, Test Coverage, and PR Checklist sections from the template. Add the template sections with a short issue/solution summary, the relevant tests or waiver validation, and the checklist confirmation.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and clearly describes the waived QA CI test entry added in this PR.
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.

@jieli-matrix

Copy link
Copy Markdown
Collaborator

/bot skip --comment "Only waive tests"

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #59164 [ skip ] triggered by Bot. Commit: d229fed Link to invocation

@jieli-matrix
jieli-matrix disabled auto-merge July 14, 2026 09:08
@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

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

Link to invocation

@jieli-matrix

Copy link
Copy Markdown
Collaborator

Close as already fixed.

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