Skip to content

[None][infra] Waive 4 failed cases for main in post-merge#12973

Merged
xinhe-nv merged 4 commits intoNVIDIA:mainfrom
xinhe-nv:trtllm-ci-report/waive-20260413-020928
Apr 13, 2026
Merged

[None][infra] Waive 4 failed cases for main in post-merge#12973
xinhe-nv merged 4 commits intoNVIDIA:mainfrom
xinhe-nv:trtllm-ci-report/waive-20260413-020928

Conversation

@xinhe-nv
Copy link
Copy Markdown
Collaborator

@xinhe-nv xinhe-nv commented Apr 13, 2026

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by qa@nvidia.com)
Target branch: main
Bug(s): 5981293, 6069686, 6069790

Waive entries added

accuracy/test_llm_api_pytorch.py::TestKimiK2::test_nvfp4[4gpus] SKIP (https://nvbugs/6069790)
accuracy/test_llm_api_pytorch.py::TestGLM4_5Air::test_nvfp4_2_model_mtp[2model_trtllm] SKIP (https://nvbugs/5981293)
accuracy/test_llm_api_pytorch.py::TestGLM4_5Air::test_nvfp4_multi_gpus[throughput] SKIP (https://nvbugs/5981293)
disaggregated/test_disaggregated.py::test_disaggregated_trtllm_sampler[TinyLlama-1.1B-Chat-v1.0] SKIP (https://nvbugs/6069686)

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

Summary by CodeRabbit

Tests

  • Added four new test case waivers to support test suite optimization.

Bug(s): 5981293, 6069686, 6069790
Requested by: qa@nvidia.com

Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
@xinhe-nv xinhe-nv force-pushed the trtllm-ci-report/waive-20260413-020928 branch from b53b495 to dfda7e0 Compare April 13, 2026 02:10
@xinhe-nv xinhe-nv enabled auto-merge (squash) April 13, 2026 02:10
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 13, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 3d956184-fd67-44de-9cec-c6e0536c2066

📥 Commits

Reviewing files that changed from the base of the PR and between 28cf4f5 and dfda7e0.

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

📝 Walkthrough

Walkthrough

Added four new SKIP test waivers to the test configuration file, covering NVFP4 accuracy tests for KimiK2 and GLM4_5Air models, plus one TRTLlm sampler test. No existing waiver entries were modified or removed.

Changes

Cohort / File(s) Summary
Test Waivers
tests/integration/test_lists/waives.txt
Added four new SKIP entries for NVFP4 PyTorch accuracy tests (TestKimiK2::test_nvfp4[4gpus], TestGLM4_5Air::test_nvfp4_2_model_mtp[2model_trtllm], TestGLM4_5Air::test_nvfp4_multi_gpus[throughput]) and one TRTLlm sampler test (test_disaggregated_trtllm_sampler[TinyLlama-1.1B-Chat-v1.0]).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • PR #11929: Adds SKIP waivers for failing nvfp4 PyTorch tests in the same waives.txt file with similar test variants.
  • PR #11968: Augments the test waivers list by adding SKIP entries to tests/integration/test_lists/waives.txt.
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description is auto-generated and includes essential information (waive entries with bug references), but lacks the structured sections from the repository template (Description, Test Coverage, PR Checklist). Consider whether auto-generated waive PRs require full template compliance. If yes, add Description and Test Coverage sections; if auto-generation is a valid exception, document this policy.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: waiving 4 failed test cases for the main branch in post-merge.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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.

@xinhe-nv
Copy link
Copy Markdown
Collaborator Author

/bot run --stage-list ""

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #42918 [ run ] triggered by Bot. Commit: dfda7e0 Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #42918 [ run ] completed with state SUCCESS. Commit: dfda7e0
/LLM/main/L0_MergeRequest_PR pipeline #33578 (Partly Tested) completed with status: 'SUCCESS'

CI Report

Link to invocation

Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
@xinhe-nv
Copy link
Copy Markdown
Collaborator Author

/bot reuse-pipeline

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #42988 [ reuse-pipeline ] triggered by Bot. Commit: 16742c1 Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #42988 [ reuse-pipeline ] completed with state SUCCESS. Commit: 16742c1
Reusing PR_Github #42918 (Partly Tested) for commit 16742c1

Link to invocation

@xinhe-nv xinhe-nv merged commit a7c4ee0 into NVIDIA:main Apr 13, 2026
5 checks passed
@xinhe-nv xinhe-nv deleted the trtllm-ci-report/waive-20260413-020928 branch April 13, 2026 08:13
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