chore(beep boop 🤖): Bump uv.lock (main, mcore-dev) (2026-03-24)#2969
chore(beep boop 🤖): Bump uv.lock (main, mcore-dev) (2026-03-24)#2969svcnvidia-nemo-ci wants to merge 1 commit intomainfrom
uv.lock (main, mcore-dev) (2026-03-24)#2969Conversation
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
|
/ok to test 9b62ea5 |
📝 WalkthroughWalkthroughThe Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 2 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@3rdparty/Megatron-LM`:
- Line 1: The submodule at 3rdparty/Megatron-LM references a non-existent commit
(c72c4599012297cfbd1d57e006b544478b6bbf78); verify and fix the submodule pointer
by ensuring the commit hash is correct and pushed to the upstream Megatron-LM
repo, or update the submodule reference to a valid commit (edit the gitlink in
the superproject index/.gitmodules and run git add on the submodule, or use git
submodule update --init --remote to point to a valid branch/commit), then commit
the corrected submodule update so CI can successfully checkout Megatron-LM.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 0df952cc-09f4-4ec7-aa49-931f3d5cc140
⛔ Files ignored due to path filters (1)
uv.lockis excluded by!**/*.lock
📒 Files selected for processing (1)
3rdparty/Megatron-LM
| @@ -1 +1 @@ | |||
| Subproject commit 8f7fbe78609de242630ca4808686bd309694dcab | |||
| Subproject commit c72c4599012297cfbd1d57e006b544478b6bbf78 | |||
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Description: Verify the new submodule commit and show the commit log between old and new references.
OLD_COMMIT="8f7fbe78609de242630ca4808686bd309694dcab"
NEW_COMMIT="c72c4599012297cfbd1d57e006b544478b6bbf78"
cd 3rdparty/Megatron-LM || exit 1
# Verify the new commit exists
echo "=== Verifying new commit exists ==="
git cat-file -t "$NEW_COMMIT" 2>/dev/null && echo "Commit $NEW_COMMIT exists" || echo "ERROR: Commit $NEW_COMMIT not found"
# Show commits between old and new
echo ""
echo "=== Commits between old and new submodule references ==="
git log --oneline "$OLD_COMMIT".."$NEW_COMMIT" 2>/dev/null | head -30
# Show summary of file changes
echo ""
echo "=== Summary of changed files ==="
git diff --stat "$OLD_COMMIT".."$NEW_COMMIT" 2>/dev/null | tail -20Repository: NVIDIA-NeMo/Megatron-Bridge
Length of output: 263
The submodule commit reference does not exist and must be verified before merging.
The new commit c72c4599012297cfbd1d57e006b544478b6bbf78 cannot be found in the Megatron-LM repository. Verify that:
- The commit hash is correct
- The commit has been pushed to the upstream repository
- The submodule is correctly configured to reference the intended commit
Attempting to check out this submodule reference will fail without a valid commit hash.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@3rdparty/Megatron-LM` at line 1, The submodule at 3rdparty/Megatron-LM
references a non-existent commit (c72c4599012297cfbd1d57e006b544478b6bbf78);
verify and fix the submodule pointer by ensuring the commit hash is correct and
pushed to the upstream Megatron-LM repo, or update the submodule reference to a
valid commit (edit the gitlink in the superproject index/.gitmodules and run git
add on the submodule, or use git submodule update --init --remote to point to a
valid branch/commit), then commit the corrected submodule update so CI can
successfully checkout Megatron-LM.
|
Closing stale bot PR. |
🚀 PR to bump
uv.lockinmain.🤖 This PR will be merged automatically once CI passes.
Summary by CodeRabbit