Skip to content

chore(deps): AUT-1080 pin merged NeMo Run cancellation fix - #6110

Merged
ko3n1g merged 2 commits into
NVIDIA:mainfrom
svcnemo-autobot:chore/pin-merged-nemo-run
Jul 29, 2026
Merged

chore(deps): AUT-1080 pin merged NeMo Run cancellation fix#6110
ko3n1g merged 2 commits into
NVIDIA:mainfrom
svcnemo-autobot:chore/pin-merged-nemo-run

Conversation

@svcnemo-autobot

Copy link
Copy Markdown
Collaborator

Background

Megatron-LM temporarily pinned NeMo Run PR #570’s open head while validating live cancellation. That PR has now merged as e3935393a290aed1822af52139b4b8ee270fed1f.

What changed

  • Replace the temporary NeMo Run head pin with the canonical merge commit.
  • Regenerate uv.lock using the repository’s uv 0.7.2 toolchain.

Details

flowchart LR
    R[NeMo Run PR #570] --> M[Merged commit]
    M --> P[Megatron-LM pyproject]
    P --> L[Regenerated uv.lock]
Loading

Lock regeneration also normalizes torch dependency markers to uv 0.7.2 output.

Tested

  • uv lock
  • uv lock --check
  • git diff --check
  • pytest -q tests/test_utils/test_launch_nemo_run_workload.py — 4 passed

Signed-off-by: svcnemo-autobot <svcnemo-autobot@nvidia.com>
Signed-off-by: svcnemo-autobot <svcnemo-autobot@nvidia.com>
@svcnemo-autobot
svcnemo-autobot requested a review from ko3n1g July 28, 2026 23:24
@svcnvidia-nemo-ci
svcnvidia-nemo-ci marked this pull request as draft July 28, 2026 23:25
@github-actions

Copy link
Copy Markdown
Contributor

This PR has been automatically converted to draft because all PRs must start as drafts.

When you are ready for review, click Ready for Review to begin the review process. This will:

  1. Add the oncall reviewer (optional reviewer)
  2. Add required review teams based on your changes

See the contribution guide for more details.

@copy-pr-bot

copy-pr-bot Bot commented Jul 28, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@ko3n1g
ko3n1g marked this pull request as ready for review July 28, 2026 23:26
@svcnvidia-nemo-ci
svcnvidia-nemo-ci requested a review from a team July 28, 2026 23:26
@ko3n1g
ko3n1g enabled auto-merge July 28, 2026 23:27
@svcnemo-autobot

Copy link
Copy Markdown
Collaborator Author

/ok to test 3994a5b

@svcnemo-autobot

Copy link
Copy Markdown
Collaborator Author

CI run 30407911285 is waiting on approval for the protected test environment. Could @NVIDIA/mcore-automation approve the pending deployment?

@ko3n1g
ko3n1g added this pull request to the merge queue Jul 29, 2026
@svcnvidia-nemo-ci

Copy link
Copy Markdown
Contributor

🔄 Merge queue validation started!

You can track the progress here: https://github.com/NVIDIA/Megatron-LM/actions/runs/30448720593

@svcnvidia-nemo-ci

Copy link
Copy Markdown
Contributor

🔄 Merge queue validation started!

You can track the progress here: https://github.com/NVIDIA/Megatron-LM/actions/runs/30454176245

@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 29, 2026
@svcnvidia-nemo-ci
svcnvidia-nemo-ci added this pull request to the merge queue Jul 29, 2026
@svcnvidia-nemo-ci

Copy link
Copy Markdown
Contributor

🔄 Merge queue validation started!

You can track the progress here: https://github.com/NVIDIA/Megatron-LM/actions/runs/30470428970

@svcnvidia-nemo-ci

Copy link
Copy Markdown
Contributor

🔄 Merge queue validation started!

You can track the progress here: https://github.com/NVIDIA/Megatron-LM/actions/runs/30473490423

@svcnvidia-nemo-ci

Copy link
Copy Markdown
Contributor

🔄 Merge queue validation started!

You can track the progress here: https://github.com/NVIDIA/Megatron-LM/actions/runs/30479289096

@ko3n1g
ko3n1g removed this pull request from the merge queue due to a manual request Jul 29, 2026
@ko3n1g
ko3n1g merged commit 54045a5 into NVIDIA:main Jul 29, 2026
89 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants