Skip to content

[https://nvbugs/6120535][chore] Unwaive DeepSeek V3.2 disaggregated serving test#16515

Open
peihu-nv wants to merge 1 commit into
NVIDIA:mainfrom
peihu-nv:ph/unwaive-6120535
Open

[https://nvbugs/6120535][chore] Unwaive DeepSeek V3.2 disaggregated serving test#16515
peihu-nv wants to merge 1 commit into
NVIDIA:mainfrom
peihu-nv:ph/unwaive-6120535

Conversation

@peihu-nv

@peihu-nv peihu-nv commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Tests
    • Removed an obsolete waiver for the disaggregated serving accuracy test, allowing it to run in the integration test suite.

Description

Re-enable accuracy/test_disaggregated_serving.py::TestDeepSeekV32Exp::test_auto_dtype[False] by removing its stale waiver.

Test Coverage

PR Checklist

Please review the following before submitting this PR:

  • The change only removes one stale test waiver; no product code or API changes are included.

  • No new dependencies, documentation changes, CODEOWNERS changes, or architecture updates are required.

  • The PR follows the TensorRT-LLM coding and contribution guidelines.

  • 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, comment /bot help.

Signed-off-by: peihengh <259410613+peihu-nv@users.noreply.github.com>
@peihu-nv

Copy link
Copy Markdown
Collaborator Author

/bot run --extra-stage "DGX_B200-8_GPUs-PyTorch-1"

@peihu-nv

Copy link
Copy Markdown
Collaborator Author

/bot run

@peihu-nv

Copy link
Copy Markdown
Collaborator Author

/bot run --extra-stage "DGX_B200-8_GPUs-PyTorch-1"

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #59998 [ run ] triggered by Bot. Commit: 5edc1ad Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #59998 [ run ] completed with state FAILURE. Commit: 5edc1ad
/LLM/main/L0_MergeRequest_PR pipeline #48391 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

@peihu-nv

Copy link
Copy Markdown
Collaborator Author

/bot run

1 similar comment
@peihu-nv

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #60046 [ run ] triggered by Bot. Commit: 5edc1ad Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #60046 [ run ] completed with state SUCCESS. Commit: 5edc1ad
/LLM/main/L0_MergeRequest_PR pipeline #48435 completed with status: 'SUCCESS'

CI Report

Link to invocation

@peihu-nv
peihu-nv marked this pull request as ready for review July 18, 2026 00:29
@peihu-nv
peihu-nv requested review from a team as code owners July 18, 2026 00:29
@coderabbitai

coderabbitai Bot commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The integration waiver list removes the skip entry for TestDeepSeekV32Exp::test_auto_dtype[False]; all other entries remain unchanged.

Changes

Waiver list update

Layer / File(s) Summary
Remove DeepSeek V32 waiver
tests/integration/test_lists/waives.txt
Removes the waiver entry for accuracy/test_disaggregated_serving.py::TestDeepSeekV32Exp::test_auto_dtype[False].

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

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
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.
Title check ✅ Passed The title matches the change and follows the repository's ticket/type/summary format.
Description check ✅ Passed The PR description follows the template and explains the waiver removal; the empty Test Coverage section is acceptable for this small change.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@peihu-nv peihu-nv changed the title [https://nvbugs/6120535][fix] Re-enable DeepSeek V3.2 disaggregated serving test [https://nvbugs/6120535][chore] Unwaive DeepSeek V3.2 disaggregated serving test Jul 18, 2026
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.

2 participants