Skip to content

[https://nvbugs/6422432][fix] Unwaive testcase - #17947

Merged
YihuiLu512 merged 1 commit into
NVIDIA:mainfrom
YihuiLu512:dev-fix-15-J14108-N6422432
Aug 26, 2026
Merged

[https://nvbugs/6422432][fix] Unwaive testcase#17947
YihuiLu512 merged 1 commit into
NVIDIA:mainfrom
YihuiLu512:dev-fix-15-J14108-N6422432

Conversation

@YihuiLu512

@YihuiLu512 YihuiLu512 commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Dev Engineer Review

  • Updated tests/integration/test_lists/waives.txt.
  • Removed obsolete DeepSeek R1 latency and DeepSeekV3Lite NVFP4 waivers.
  • Added waivers for GLM52 NVFP4, Qwen softmax attention, LTX2 CUDA-graph TRTLLM, DGX B200 performance alignment, Exaone attention backends, Qwen3 LoRA, standalone packaging, Kimi K3 generation performance, and related performance scenarios.
  • No API, performance, or error-handling code changed.
  • The available diff output did not show the complete entry-level changes. Verify waiver format, bug references, test paths, scope, and duplicate entries.

QA Engineer Review

  • Modified file: tests/integration/test_lists/waives.txt.
  • Added waivers for the GLM52, Qwen, LTX2, DGX B200, Exaone, Qwen3, standalone packaging, Kimi K3, and related performance scenarios.
  • Removed DeepSeek R1 latency and DeepSeekV3Lite NVFP4 waivers.
  • No test-db/ or qa/ files were modified.
  • Verdict: needs follow-up because CBTS coverage data is unavailable.

Description

Test Coverage

/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2"

PR Checklist

Please review the following before submitting your PR:

  • PR description clearly explains what and why. If using CodeRabbit's summary, please make sure it makes sense.

  • PR Follows TRT-LLM CODING GUIDELINES to the best of your knowledge.

  • Test cases are provided for new code paths (see test instructions)

  • If PR introduces API changes, an appropriate PR label is added - either api-compatible or api-breaking. For api-breaking, include BREAKING in the PR title.

  • Any new dependencies have been scanned for license and vulnerabilities

  • CODEOWNERS updated if ownership changes

  • Documentation updated as needed

  • Update tava architecture diagram if there is a significant design change in PR.

  • The reviewers assigned automatically/manually are appropriate for the PR.

  • Please check this after reviewing the above items as appropriate for this PR.

GitHub Bot Help

To see a list of available CI bot commands, please comment /bot help.

@YihuiLu512

Copy link
Copy Markdown
Collaborator Author

/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2"

@YihuiLu512
YihuiLu512 enabled auto-merge (squash) August 19, 2026 03:55
@coderabbitai

coderabbitai Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review

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: aed7f500-5694-450d-9278-eea8bd454441

📥 Commits

Reviewing files that changed from the base of the PR and between 02d0f85 and 1496608.

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

Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review.


Walkthrough

The change updates tests/integration/test_lists/waives.txt with current model, hardware, packaging, and performance waivers. It also removes obsolete DeepSeek waiver entries.

Changes

Integration test waiver maintenance

Layer / File(s) Summary
Model and serving waivers
tests/integration/test_lists/waives.txt
Adds waivers for GLM52, Qwen3 softmax attention, Exaone attention backends, and Qwen3 FP8 LoRA tests. Removes obsolete DeepSeek waivers.
Visual-generation and hardware waivers
tests/integration/test_lists/waives.txt
Adds waivers for LTX2 CUDA-graph TRTLLM tests and DGX B200 layer-wise benchmark performance.
Performance and packaging waivers
tests/integration/test_lists/waives.txt
Adds waivers for GB300 DeepSeek scenarios, standalone-package unit tests, and Kimi K3 generation performance.

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

Merge Risk: 🔵 Low · up to 14966

The PR re-enables a previously waived testcase. It is mergeable with owner awareness, but the affected GB200, DGX B300, and RTXPro6000D post-merge stages should be confirmed to ensure the waiver removal applies to the intended scope.

Possibly related PRs

Suggested reviewers: bowenfu, tensorrt-cicd, trtllm-agent

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description includes the required template and a test command, but the Description section does not explain the issue or solution. The test coverage section also does not identify the relevant tes… Add a short Description that states which testcase is being unwaived and why. Expand Test Coverage with the relevant test or waiver changes and explain how the listed CI stages validate the change.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title follows the required NVBugs and type format and clearly identifies the change as a testcase unwaive fix.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
Full details: Description check

Explanation

The description includes the required template and a test command, but the Description section does not explain the issue or solution. The test coverage section also does not identify the relevant tests or expected validation.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files.

  • Fix all pre-merge checks with AI
✨ 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 #67268 [ run ] triggered by Bot. Commit: a8031c3 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #67268 [ run ] completed with state SUCCESS. Commit: a8031c3
/LLM/main/L0_MergeRequest_PR pipeline #54793 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@YihuiLu512

Copy link
Copy Markdown
Collaborator Author

@QiJune Please add the following label so that I can verify whether the bug is reproducible: ci: post-merge approved.

@YihuiLu512

Copy link
Copy Markdown
Collaborator Author

/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #67453 [ run ] triggered by Bot. Commit: a8031c3 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #67453 [ run ] completed with state SUCCESS. Commit: a8031c3
/LLM/main/L0_MergeRequest_PR pipeline #54954 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@YihuiLu512 YihuiLu512 added the ci: post-merge approved Approved by TRT-LLM CI approvers for broad post-merge CI requests label Aug 20, 2026
@github-actions github-actions Bot removed the ci: post-merge approved Approved by TRT-LLM CI approvers for broad post-merge CI requests label Aug 20, 2026
@github-actions

Copy link
Copy Markdown

Removed the "ci: post-merge approved" label because @YihuiLu512 could not be verified as an active member of NVIDIA/trt-llm-ci-approvers. Ask a member of that team to apply it.

@YihuiLu512

Copy link
Copy Markdown
Collaborator Author

/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast

@YihuiLu512

Copy link
Copy Markdown
Collaborator Author

/bot kill

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #67635 [ kill ] triggered by Bot. Commit: a8031c3 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #67635 [ kill ] completed with state SUCCESS. Commit: a8031c3
Successfully killed previous jobs for commit a8031c3

Link to invocation

@QiJune QiJune added the ci: post-merge approved Approved by TRT-LLM CI approvers for broad post-merge CI requests label Aug 20, 2026
@YihuiLu512

Copy link
Copy Markdown
Collaborator Author

/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #67653 [ run ] triggered by Bot. Commit: a8031c3 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #67653 [ run ] completed with state FAILURE. Commit: a8031c3
/LLM/main/L0_MergeRequest_PR pipeline #55140 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@YihuiLu512

Copy link
Copy Markdown
Collaborator Author

/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #67748 [ run ] triggered by Bot. Commit: a8031c3 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #67748 [ run ] completed with state FAILURE. Commit: a8031c3
/LLM/main/L0_MergeRequest_PR pipeline #55227 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@YihuiLu512

Copy link
Copy Markdown
Collaborator Author

/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #68062 [ run ] triggered by Bot. Commit: a8031c3 Link to invocation

@YihuiLu512

Copy link
Copy Markdown
Collaborator Author

/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #68646 [ run ] completed with state ABORTED. Commit: a8031c3

Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #68666 [ run ] completed with state FAILURE. Commit: a022956
/LLM/main/L0_MergeRequest_PR pipeline #56070 completed with status: 'UNSTABLE'

CI Report

⚠️ Multi-GPU Label Required:
Multi-GPU tests require the ci: full pre-merge approved label on this PR. Ask a member of NVIDIA/trt-llm-ci-approvers to add the label, then re-trigger CI with the same bot command (no rebase needed).

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

Link to invocation

@YihuiLu512

Copy link
Copy Markdown
Collaborator Author

/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #68748 [ run ] triggered by Bot. Commit: a022956 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #68748 [ run ] completed with state SUCCESS. Commit: a022956
/LLM/main/L0_MergeRequest_PR pipeline #56147 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@YihuiLu512
YihuiLu512 force-pushed the dev-fix-15-J14108-N6422432 branch from a022956 to 3aa54cc Compare August 25, 2026 03:22
@YihuiLu512

Copy link
Copy Markdown
Collaborator Author

/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #68989 [ run ] triggered by Bot. Commit: 3aa54cc Link to invocation

@YihuiLu512

Copy link
Copy Markdown
Collaborator Author

/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #68989 [ run ] completed with state FAILURE. Commit: 3aa54cc
/LLM/main/L0_MergeRequest_PR pipeline #56365 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #68996 [ run ] triggered by Bot. Commit: 3aa54cc Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #68996 [ run ] completed with state FAILURE. Commit: 3aa54cc
/LLM/main/L0_MergeRequest_PR pipeline #56372 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@YihuiLu512
YihuiLu512 force-pushed the dev-fix-15-J14108-N6422432 branch from 3aa54cc to 02d0f85 Compare August 26, 2026 01:05
@YihuiLu512

Copy link
Copy Markdown
Collaborator Author

/bot run --extra-stage "DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #69254 [ run ] triggered by Bot. Commit: 02d0f85 Link to invocation

@YihuiLu512

Copy link
Copy Markdown
Collaborator Author

/bot kill

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #69324 [ kill ] triggered by Bot. Commit: 02d0f85 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #69254 [ run ] completed with state ABORTED. Commit: 02d0f85
/LLM/main/L0_MergeRequest_PR pipeline #56612 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #69324 [ kill ] completed with state SUCCESS. Commit: 02d0f85
Successfully killed previous jobs for commit 02d0f85

Link to invocation

@YihuiLu512

Copy link
Copy Markdown
Collaborator Author

/bot run --stage-list "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #69330 [ run ] triggered by Bot. Commit: 02d0f85 Link to invocation

Signed-off-by: Yihui Lu <269394165+YihuiLu512@users.noreply.github.com>
@YihuiLu512
YihuiLu512 force-pushed the dev-fix-15-J14108-N6422432 branch from 02d0f85 to 1496608 Compare August 26, 2026 08:52
@YihuiLu512

Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #69399 [ run ] triggered by Bot. Commit: 1496608 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #69330 [ run ] completed with state ABORTED. Commit: 02d0f85

Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #69399 [ run ] completed with state SUCCESS. Commit: 1496608
/LLM/main/L0_MergeRequest_PR pipeline #56736 completed with status: 'SUCCESS'

CI Report

Link to invocation

@YihuiLu512
YihuiLu512 merged commit 8bde010 into NVIDIA:main Aug 26, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci: full pre-merge approved ci: post-merge approved Approved by TRT-LLM CI approvers for broad post-merge CI requests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants