Skip to content

Conversation

@nvchenghaoz
Copy link
Collaborator

@nvchenghaoz nvchenghaoz commented Nov 20, 2025

Summary by CodeRabbit

  • Tests
    • Updated test configurations across multiple hardware profiles (b200, dgx_h200, h100)
    • Modified and added AutoDeploy test entries to expand test coverage for different model configurations

Signed-off-by: Chenghao Zhang <[email protected]>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 20, 2025

📝 Walkthrough

Walkthrough

Test configuration updates across three test list files replacing Llama3.1 8B auto dtype tests with Nemotron MOE precision-specific tests (fp8 and bf16 variants). No logic or behavioral changes; only test case selections updated.

Changes

Cohort / File(s) Summary
Test list file updates
tests/integration/test_lists/test-db/l0_b200.yml, tests/integration/test_lists/test-db/l0_dgx_h200.yml, tests/integration/test_lists/test-db/l0_h100.yml
Replaced TestLlama3_1_8B::test_auto_dtype entries with TestNemotronMOE::test_fp8 and TestNemotronMOE::test_bf16 entries across AutoDeploy test sections. l0_h100.yml adds both new test entries; l0_dgx_h200.yml updates two AutoDeploy blocks (PyTorch and TRT); l0_b200.yml updates one pre-merge AutoDeploy block.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

  • Test configuration updates follow a consistent, homogeneous pattern across all three files
  • Changes are simple replacements and additions of test case references
  • No logic, code, or structural modifications present

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description is minimal, containing only an auto-generated summary by CodeRabbit. Required sections like explicit description, test coverage details, and PR checklist confirmation are incomplete. Provide a clear explanation of why Nemotron MOE tests are being added, which specific test cases were modified/added, and confirm the PR checklist items have been reviewed.
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title clearly describes the main change: adding Nemotron MOE tests to the AutoDeploy CI configuration across multiple hardware profiles.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@nvchenghaoz nvchenghaoz changed the title [None][chore] AutoDeploy: Add the test to CI [None][chore] AutoDeploy: Add the Nemotron MOE to CI Nov 20, 2025
@nvchenghaoz
Copy link
Collaborator Author

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #25154 [ run ] triggered by Bot. Commit: 2e6ef7a

@tensorrt-cicd
Copy link
Collaborator

PR_Github #25154 [ run ] completed with state SUCCESS. Commit: 2e6ef7a
/LLM/main/L0_MergeRequest_PR pipeline #19018 completed with status: 'FAILURE'

@nvchenghaoz
Copy link
Collaborator Author

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #25229 [ run ] triggered by Bot. Commit: 2e6ef7a

@tensorrt-cicd
Copy link
Collaborator

PR_Github #25229 [ run ] completed with state SUCCESS. Commit: 2e6ef7a
/LLM/main/L0_MergeRequest_PR pipeline #19081 completed with status: 'FAILURE'

@nvchenghaoz
Copy link
Collaborator Author

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #25234 [ run ] triggered by Bot. Commit: 2e6ef7a

@tensorrt-cicd
Copy link
Collaborator

PR_Github #25234 [ run ] completed with state SUCCESS. Commit: 2e6ef7a
/LLM/main/L0_MergeRequest_PR pipeline #19085 completed with status: 'FAILURE'

@nvchenghaoz
Copy link
Collaborator Author

Disable the auto merge as it is blocked the #9339 .

@nvchenghaoz
Copy link
Collaborator Author

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #25385 [ run ] triggered by Bot. Commit: 2e6ef7a

@tensorrt-cicd
Copy link
Collaborator

PR_Github #25385 [ run ] completed with state SUCCESS. Commit: 2e6ef7a
/LLM/main/L0_MergeRequest_PR pipeline #19203 completed with status: 'FAILURE'

@nvchenghaoz
Copy link
Collaborator Author

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #25392 [ run ] triggered by Bot. Commit: 2e6ef7a

@tensorrt-cicd
Copy link
Collaborator

PR_Github #25392 [ run ] completed with state FAILURE. Commit: 2e6ef7a
/LLM/main/L0_MergeRequest_PR pipeline #19210 completed with status: 'FAILURE'

@nvchenghaoz
Copy link
Collaborator Author

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #25404 [ run ] triggered by Bot. Commit: eb52a91

@github-project-automation github-project-automation bot moved this from Backlog to In review in AutoDeploy Board Nov 22, 2025
@tensorrt-cicd
Copy link
Collaborator

PR_Github #25404 [ run ] completed with state SUCCESS. Commit: eb52a91
/LLM/main/L0_MergeRequest_PR pipeline #19221 completed with status: 'FAILURE'

@suyoggupta
Copy link
Collaborator

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #25410 [ run ] triggered by Bot. Commit: b9f81ad

@tensorrt-cicd
Copy link
Collaborator

PR_Github #25410 [ run ] completed with state SUCCESS. Commit: b9f81ad
/LLM/main/L0_MergeRequest_PR pipeline #19226 completed with status: 'FAILURE'

@suyoggupta
Copy link
Collaborator

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #25430 [ run ] triggered by Bot. Commit: b9f81ad

@tensorrt-cicd
Copy link
Collaborator

PR_Github #25430 [ run ] completed with state SUCCESS. Commit: b9f81ad
/LLM/main/L0_MergeRequest_PR pipeline #19244 completed with status: 'FAILURE'

@suyoggupta
Copy link
Collaborator

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #25438 [ run ] triggered by Bot. Commit: b9f81ad

@tensorrt-cicd
Copy link
Collaborator

PR_Github #25438 [ run ] completed with state SUCCESS. Commit: b9f81ad
/LLM/main/L0_MergeRequest_PR pipeline #19252 completed with status: 'SUCCESS'

@nvchenghaoz nvchenghaoz merged commit e1c9aa7 into NVIDIA:main Nov 23, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in AutoDeploy Board Nov 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants