Skip to content

[None][infra] Waive 19 failed cases for main in post-merge 2884 - #17272

Merged
mzweilz merged 1 commit into
NVIDIA:mainfrom
trtllm-agent:trtllm-ci-report/waive-20260805-022701
Aug 5, 2026
Merged

[None][infra] Waive 19 failed cases for main in post-merge 2884#17272
mzweilz merged 1 commit into
NVIDIA:mainfrom
trtllm-agent:trtllm-ci-report/waive-20260805-022701

Conversation

@trtllm-agent

@trtllm-agent trtllm-agent commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by @mzweilz)
Target branch: main
Bug(s): 6550708, 6555875, 6561558, 6561563, 6561566

Waive entries added

accuracy/test_disaggregated_serving.py::TestNemotron3Super120B::test_auto_dtype[mtp_nextn=3-block_reuse=True-use_py_transceiver=False] SKIP (https://nvbugs/6561563)
accuracy/test_llm_api_pytorch.py::TestQwen3_30B_A3B::test_nvfp4[tep4_latency_moe_cutlass-torch_compile=True] SKIP (https://nvbugs/6561558)
perf/test_perf_sanity.py::test_e2e[disagg_upload-gen_only-b200_deepseek-r1-fp4_8k1k_con1_ctx1_dep4_gen1_tep8_eplb0_mtp3_ccb-NIXL] SKIP (https://nvbugs/6561566)
perf/test_perf_sanity.py::test_e2e[disagg_upload-gen_only-b200_deepseek-r1-fp4_8k1k_con256_ctx1_dep4_gen1_dep8_eplb0_mtp1_ccb-NIXL] SKIP (https://nvbugs/6561566)
perf/test_perf_sanity.py::test_e2e[disagg_upload-gen_only-gb200_deepseek-r1-fp4_8k1k_con1024_ctx1_dep4_gen1_dep32_eplb0_mtp3_ccb-NIXL] SKIP (https://nvbugs/6561566)
perf/test_perf_sanity.py::test_e2e[disagg_upload-gen_only-gb200_deepseek-r1-fp4_8k1k_con1_ctx1_dep4_gen1_tep8_eplb0_mtp3_ccb-NIXL] SKIP (https://nvbugs/6561566)
perf/test_perf_sanity.py::test_e2e[disagg_upload-gen_only-gb200_deepseek-v32-fp4_32k4k_con256_ctx1_dep4_gen1_dep32_eplb0_mtp3_ccb-NIXL] SKIP (https://nvbugs/6561566)
perf/test_perf_sanity.py::test_e2e[disagg_upload-gen_only-gb200_deepseek-v32-fp4_8k1k_con1_ctx1_dep4_gen1_tep8_eplb0_mtp3_ccb-NIXL] SKIP (https://nvbugs/6561566)
perf/test_perf_sanity.py::test_e2e[disagg_upload-gen_only-gb200_gpt-oss-120b-fp4_8k1k_con1024_ctx1_tp1_gen1_tp4_eplb0_mtp0_ccb-NIXL] SKIP (https://nvbugs/6561566)
perf/test_perf_sanity.py::test_e2e[disagg_upload-gen_only-gb200_gpt-oss-120b-fp4_8k1k_con128_ctx1_tp1_gen1_tp4_eplb0_mtp0_ccb-NIXL] SKIP (https://nvbugs/6561566)
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/6561566)
perf/test_perf_sanity.py::test_e2e[disagg_upload-gen_only-gb300_glm-5-fp4_8k1k_con1024_ctx1_dep2_gen1_dep8_eplb256_mtp1_ccb-NIXL] SKIP (https://nvbugs/6561566)
perf/test_perf_sanity.py::test_e2e[disagg_upload-gen_only-gb300_glm-5-fp4_8k1k_con1_ctx1_dep2_gen1_tep8_eplb0_mtp3_ccb-NIXL] SKIP (https://nvbugs/6561566)
perf/test_perf_sanity.py::test_e2e[disagg_upload-gen_only-gb300_kimi-k25-thinking-fp4_8k1k_con4096_ctx1_dep4_gen1_dep16_eplb0_mtp0_ccb-NIXL] SKIP (https://nvbugs/6561566)
unittest/_torch/visual_gen/test_cosmos3_distilled.py::TestDistilledConditioningAnchor::test_every_forward_sees_clean_conditioned_frame SKIP (https://nvbugs/6550708)
unittest/_torch/visual_gen/test_cosmos3_distilled.py::TestDistilledConditioningAnchor::test_without_anchor_the_conditioned_frame_drifts SKIP (https://nvbugs/6550708)
unittest/_torch/visual_gen/test_cosmos3_distilled.py::TestDistilledDenoiseLoop::test_guidance_one_single_forward_per_step SKIP (https://nvbugs/6550708)
unittest/_torch/visual_gen/test_cosmos3_distilled.py::TestDistilledDenoiseLoop::test_scheduler_step_kwargs_reach_every_step SKIP (https://nvbugs/6550708)
unittest/llmapi/test_llm.py::test_openai_completion_list_prompt_stream_reuses_stream_metadata SKIP (https://nvbugs/6555875)

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

Summary

  • Added 19 post-merge waiver entries to tests/integration/test_lists/waives.txt.
  • Covered Nemotron3Super120B, Qwen3 NVFP4, disaggregated performance, Cosmos3 distilled visual-generation, and OpenAI completion streaming tests.
  • Associated each waiver with an NVIDIA bug.

Dev Engineer Review

  • The change is limited to the waiver list.
  • No exported or public entities changed.
  • The entries require validation against test-list format, test paths, bug references, and duplicate entries.

QA Engineer Review

  • Modified files: tests/integration/test_lists/waives.txt.
  • Added 19 waiver entries.
  • No test code changed.
  • CBTS coverage data is unavailable.
  • Verdict: needs follow-up.

Bug(s): 6550708, 6555875, 6561558, 6561563, 6561566
Requested by: @mzweilz

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

mzweilz commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

/bot skip --comment "waive only"

@mzweilz
mzweilz enabled auto-merge (squash) August 5, 2026 02:27
@coderabbitai

coderabbitai Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

The integration waiver list adds 19 skipped test entries. The entries cover model serving, disaggregated performance, Cosmos3 visual generation, and OpenAI completion streaming scenarios. Each entry references an NVIDIA bug.

Changes

Integration test waivers

Layer / File(s) Summary
Model and performance waivers
tests/integration/test_lists/waives.txt
Adds skipped Nemotron3Super120B, Qwen3 NVFP4, and disaggregated performance scenarios for multiple hardware and model configurations.
Feature-specific waivers
tests/integration/test_lists/waives.txt
Adds skipped Cosmos3 visual-generation tests and an OpenAI completion list-prompt streaming test.

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

Possibly related PRs

Suggested reviewers: bowenfu, 2ez4bz

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the infrastructure change, waiver count, target branch, and post-merge run.
Description check ✅ Passed The description clearly explains the waiver scope, bug references, affected test cases, target branch, and review requirement.
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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@tests/integration/test_lists/waives.txt`:
- Line 14: Add the repository-standard NVIDIA copyright header at the beginning
of waives.txt, using 2026 as the copyright year, while preserving the existing
waiver entry unchanged.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: e138bf55-1e58-461b-a426-a9a80e48d124

📥 Commits

Reviewing files that changed from the base of the PR and between 91fb443 and aec658e.

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

Comment thread tests/integration/test_lists/waives.txt
@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #63903 [ skip ] triggered by Bot. Commit: aec658e Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

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

Link to invocation

@BowenFu BowenFu left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The waiver entries are scoped to exact test nodes and linked to follow-up NVBugs. Approved.

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.

4 participants