Skip to content

[https://nvbugs/5981122][fix] Unwaive DeepSeekV3Lite python_scheduler test#12972

Merged
lancelly merged 8 commits intoNVIDIA:mainfrom
lancelly:unwaive-deepseekv3lite-python-scheduler
Apr 21, 2026
Merged

[https://nvbugs/5981122][fix] Unwaive DeepSeekV3Lite python_scheduler test#12972
lancelly merged 8 commits intoNVIDIA:mainfrom
lancelly:unwaive-deepseekv3lite-python-scheduler

Conversation

@lancelly
Copy link
Copy Markdown
Collaborator

@lancelly lancelly commented Apr 13, 2026

Summary

  • Remove waive for test_bfloat16_python_scheduler[mtp_nextn=2-...] — NVBug https://nvbugs/5981122
    Test pass consistently across 20/20 runs.

… test

Remove waive for TestDeepSeekV3Lite::test_bfloat16_python_scheduler
[mtp_nextn=2-attention_dp=True-cuda_graph=True-overlap_scheduler=True
-enable_chunked_prefill=False] as it passes consistently (20/20 runs
on H100).

Signed-off-by: Lanyu Liao <lancelly@users.noreply.github.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 13, 2026

📝 Walkthrough

Walkthrough

Removed one waived test entry from the integration test skip list that was previously excluding a specific DeepSeekV3Lite test case due to an NVIDIA bug reference.

Changes

Cohort / File(s) Summary
Test Waiver List
tests/integration/test_lists/waives.txt
Removed waived test entry for accuracy/test_llm_api_pytorch.py::TestDeepSeekV3Lite::test_bfloat16_python_scheduler[...] (nvbugs/5981122); test will no longer be skipped.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description is minimal and lacks required sections from the template (Test Coverage, PR Checklist). While it briefly explains the change, it does not follow the structured format specified. Expand the description to include Test Coverage section detailing the 20/20 test runs and confirm the PR Checklist items have been reviewed.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: removing a waiver for a specific DeepSeekV3Lite test that now passes consistently.
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.

@lancelly
Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #42910 [ run ] triggered by Bot. Commit: d9abfbb Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #42910 [ run ] completed with state SUCCESS. Commit: d9abfbb
/LLM/main/L0_MergeRequest_PR pipeline #33570 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

Link to invocation

@lancelly lancelly force-pushed the unwaive-deepseekv3lite-python-scheduler branch from 1e63990 to d9abfbb Compare April 13, 2026 06:26
@lancelly
Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #42972 [ run ] triggered by Bot. Commit: d9abfbb Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #42972 [ run ] completed with state SUCCESS. Commit: d9abfbb
/LLM/main/L0_MergeRequest_PR pipeline #33625 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

Link to invocation

@lancelly
Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43009 [ run ] triggered by Bot. Commit: 91d95b1 Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43009 [ run ] completed with state SUCCESS. Commit: 91d95b1
/LLM/main/L0_MergeRequest_PR pipeline #33659 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

Link to invocation

@lancelly
Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43250 [ run ] triggered by Bot. Commit: eebc9a4 Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43250 [ run ] completed with state DISABLED
Freeze main and open the PR merge only after CI is back to healthy https://nvidia.slack.com/archives/C059LSY62BT/p1776141760843319?thread_ts=1775985925.442509&cid=C059LSY62BT

Link to invocation

@lancelly
Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43315 [ run ] triggered by Bot. Commit: df0a1fb Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43315 [ run ] completed with state SUCCESS. Commit: df0a1fb
/LLM/main/L0_MergeRequest_PR pipeline #33857 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

Link to invocation

@lancelly
Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43365 [ run ] triggered by Bot. Commit: 8305b30 Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43365 [ run ] completed with state SUCCESS. Commit: 8305b30
/LLM/main/L0_MergeRequest_PR pipeline #33902 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

Link to invocation

@lancelly
Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43452 [ run ] triggered by Bot. Commit: 8305b30 Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43452 [ run ] completed with state SUCCESS. Commit: 8305b30
/LLM/main/L0_MergeRequest_PR pipeline #33976 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

Link to invocation

@lancelly
Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43506 [ run ] triggered by Bot. Commit: 786b5ed Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43506 [ run ] completed with state SUCCESS. Commit: 786b5ed
/LLM/main/L0_MergeRequest_PR pipeline #34020 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

Link to invocation

@lancelly
Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43601 [ run ] triggered by Bot. Commit: 8b71568 Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43601 [ run ] completed with state SUCCESS. Commit: 8b71568
/LLM/main/L0_MergeRequest_PR pipeline #34094 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

Link to invocation

@lancelly
Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43738 [ run ] triggered by Bot. Commit: 95f7d0d Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43738 [ run ] completed with state FAILURE. Commit: 95f7d0d
/LLM/main/L0_MergeRequest_PR pipeline #34221 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

Link to invocation

@lancelly
Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43867 [ run ] triggered by Bot. Commit: 95f7d0d Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43867 [ run ] completed with state FAILURE. Commit: 95f7d0d
/LLM/main/L0_MergeRequest_PR pipeline #34323 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

Link to invocation

@lancelly
Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43931 [ run ] triggered by Bot. Commit: 95f7d0d Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #43931 [ run ] completed with state SUCCESS. Commit: 95f7d0d
/LLM/main/L0_MergeRequest_PR pipeline #34380 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

Link to invocation

@lancelly
Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #44081 [ run ] triggered by Bot. Commit: 95f7d0d Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #44081 [ run ] completed with state SUCCESS. Commit: 95f7d0d
/LLM/main/L0_MergeRequest_PR pipeline #34511 completed with status: 'SUCCESS'

CI Report

Link to invocation

@lancelly lancelly requested a review from xinhe-nv April 18, 2026 14:22
@lancelly lancelly enabled auto-merge (squash) April 20, 2026 13:32
@lancelly lancelly merged commit 0ea80c7 into NVIDIA:main Apr 21, 2026
5 checks passed
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