[https://nvbugs/6428094][chore] Unwaive DeepSeekV3Lite tp2pp2 test - #17155
[https://nvbugs/6428094][chore] Unwaive DeepSeekV3Lite tp2pp2 test#17155cascade812 wants to merge 1 commit into
Conversation
Signed-off-by: Guiju Zhang <7135567+cascade812@users.noreply.github.com>
WalkthroughThe integration waiver list no longer skips the DeepSeekV3Lite TP2/PP2 BF16 4-GPU test with ChangesDeepSeekV3Lite integration test
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Comment |
BowenFu
left a comment
There was a problem hiding this comment.
Waiver removal only. The test sits in a post_merge block (l0_dgx_h200.yml:112), so the green pre-merge L0 here is not coverage for it — worth a /bot run --extra-stage "DGX_H200-4_GPUs-PyTorch-Post-Merge-1" before merge. Not blocking: nvbugs/6428094 has two independent non-repros on 4xH200 (repair-bot 07-09, engineer 07-31) against a single post-merge occurrence.
Dev Engineer Review
DeepSeekV3LiteTP2/PP2 BF16 4-GPU skip entry fromtests/integration/test_lists/waives.txt.6428094.QA Engineer Review
tests/integration/test_lists/waives.txt.DeepSeekV3LiteTP2/PP2 BF16 4-GPU entry withmtp_nextn=0, no attention DP, CUDA graphs, overlap scheduling, or Torch compilation.test-db/orqa/files were modified.Description
Test Coverage
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-compatibleorapi-breaking. Forapi-breaking, includeBREAKINGin 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.