Skip to content

[None][infra] Waive 5 failed cases for main in post-merge 2823 - #15968

Merged
mzweilz merged 1 commit into
NVIDIA:mainfrom
trtllm-agent:trtllm-ci-report/waive-20260706-083605
Jul 6, 2026
Merged

[None][infra] Waive 5 failed cases for main in post-merge 2823#15968
mzweilz merged 1 commit into
NVIDIA:mainfrom
trtllm-agent:trtllm-ci-report/waive-20260706-083605

Conversation

@trtllm-agent

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

Copy link
Copy Markdown
Collaborator

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by @mzweilz)
Target branch: main
Bug(s): 6418815, 6418822, 6418830, 6418834

Waive entries added

accuracy/test_llm_api_pytorch.py::TestQwen3_5_397B_A17B::test_nvfp4_4gpus_static_eplb[moe_backend=TRTLLM] SKIP (https://nvbugs/6418830)
examples/visual_gen/test_visual_gen.py::test_cosmos3_nano_t2i_lpips_against_golden SKIP (https://nvbugs/6418815)
perf/test_perf_sanity.py::test_e2e[disagg_upload-gen_only-gb200_qwen3-235b-fp4_8k1k_con64_ctx1_tp1_gen1_tep4_eplb0_mtp0_ccb-NIXL] SKIP (https://nvbugs/6418834)
perf/test_perf_sanity.py::test_e2e[disagg_upload-gen_only-gb300_glm-5-fp4_1k1k_con1_ctx1_dep2_gen1_tep4_eplb0_mtp3_ccb-NIXL] SKIP (https://nvbugs/6418834)
unittest/_torch/visual_gen/test_wan21_i2v_pipeline.py::TestWanI2VBatchGeneration::test_batch_prompt_shape SKIP (https://nvbugs/6418822)

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

Summary by CodeRabbit

  • Tests
    • Added new test waivers for several specific hardware and pipeline scenarios.
    • Updated existing waiver coverage for visual generation, performance sanity, and batch prompt shape checks.

Bug(s): 6418815, 6418822, 6418830, 6418834
Requested by: @mzweilz

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

mzweilz commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator

/bot skip --comment "waive only"

@mzweilz
mzweilz enabled auto-merge (squash) July 6, 2026 08:37
@coderabbitai

coderabbitai Bot commented Jul 6, 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: d8c5c65c-a1c9-4daa-8d0b-7391d91b472f

📥 Commits

Reviewing files that changed from the base of the PR and between 9727b20 and a9a8473.

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

📝 Walkthrough

Walkthrough

This PR adds four new SKIP waiver entries to the integration test waivers list, disabling specific failing tests across accuracy, visual generation, performance sanity, and unittest categories, each referencing an associated NVBugs tracking issue.

Changes

Waiver list updates

Layer / File(s) Summary
New SKIP waivers for failing tests
tests/integration/test_lists/waives.txt
Adds waivers for TestQwen3_5_397B_A17B::test_nvfp4_4gpus_static_eplb, test_cosmos3_nano_t2i_lpips_against_golden, a disagg_upload-gen_only-gb200_qwen3-235b perf-sanity case, and TestWanI2VBatchGeneration::test_batch_prompt_shape, each with an associated NVBugs reference.

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

Possibly related PRs

  • NVIDIA/TensorRT-LLM#15195: Also adds Cosmos3-related SKIP waivers to the same waiver list.
  • NVIDIA/TensorRT-LLM#15914: Overlapping waiver entries for Qwen3 MoE, GB200/GB300 gen-only perf tests, and Wan21 visual-generation tests in the same file.
  • NVIDIA/TensorRT-LLM#15825: Adds/updates SKIP entries in the same waiver file for overlapping perf-sanity and MoE multi-GPU categories.

Suggested reviewers: mzweilz, brb-nv, xinhe-nv, ZhanruiSunCh

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning It lists the waived cases and bug links, but it omits the required Description, Test Coverage, and PR Checklist sections. Add the template sections with a brief issue/solution summary, relevant test coverage, and PR checklist confirmations.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise, correctly typed as infra, and matches the main change of waiving failed cases.
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 #57738 [ skip ] triggered by Bot. Commit: a9a8473 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

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

Link to invocation

@mzweilz
mzweilz merged commit 4f1881d into NVIDIA:main Jul 6, 2026
12 checks passed
BrianLi23 pushed a commit to BrianLi23/TensorRT-LLM that referenced this pull request Jul 9, 2026
…A#15968)

Signed-off-by: trtllm-agent <296075020+trtllm-agent@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