Skip to content

ci: bump async-gym megatron test wall-clock budget to 120 min#3258

Merged
yuki-97 merged 2 commits into
mainfrom
ci/bump-nanov3-async-gym-timeout
Jul 17, 2026
Merged

ci: bump async-gym megatron test wall-clock budget to 120 min#3258
yuki-97 merged 2 commits into
mainfrom
ci/bump-nanov3-async-gym-timeout

Conversation

@shanmugamr1992

Copy link
Copy Markdown
Contributor

Summary

  • Bump NUM_MINUTES from 90 to 120 in grpo-nanov3-30BA3B-2n8g-megatron_generation-async-gym.sh.
  • Job 364261521 completes all 8 async GRPO steps (Async GRPO training complete!) but is SIGKILLed at the 90-minute Slurm wall-clock limit during teardown, before json_dump_tb_logs.py / check_metrics.py run. nemo-ci classifies exit 137 as Signal/OOM-Kill; there is no CUDA OOM in the driver log.
  • Wall-clock breakdown: ~25 min startup (30B-MoE load + CUDA-graph warmup + nemo_gym servers) + ~63 min for 8 steps leaves ~0 headroom at 90 min. PR fix(grpo): pin inference_optimized config for newer Megatron-Core images #3150 already reduced steps 10→8 but kept the 90 min budget.

Test plan

  • /ok to test <commit-sha> on the MR
  • Confirm llm_grpo_nanov3_30BA3B_2n8g_megatron_generation_async_gym finishes with metrics.json written on cw_dfw

The llm_grpo_nanov3_30BA3B_2n8g_megatron_generation_async_gym job completes
all 8 GRPO steps but is SIGKILLed at the 90-minute Slurm limit during
teardown before metrics.json is written. nemo-ci mislabels the exit-137 as
Signal/OOM-Kill; there is no CUDA OOM in the logs.

Signed-off-by: shanmugamr1992 <shanmugamr1992@gmail.com>
@shanmugamr1992
shanmugamr1992 requested a review from a team as a code owner July 16, 2026 23:56
@copy-pr-bot

copy-pr-bot Bot commented Jul 16, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@kajalj22 kajalj22 added the CI:docs Run doctest label Jul 16, 2026
@yuki-97 yuki-97 added CI:Lfast Runs a fast test suite and re-use nightly `main` container (but sync dependencies to PRs version) r0.7.0 and removed CI:Lfast Runs a fast test suite and re-use nightly `main` container (but sync dependencies to PRs version) labels Jul 17, 2026
@yuki-97

yuki-97 commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

/ok to test b6357cd

@yuki-97
yuki-97 enabled auto-merge (squash) July 17, 2026 10:02
@yuki-97

yuki-97 commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

/ok to test d0af02d

@yuki-97
yuki-97 merged commit bed45e6 into main Jul 17, 2026
52 checks passed
@yuki-97
yuki-97 deleted the ci/bump-nanov3-async-gym-timeout branch July 17, 2026 10:30
arnavk-nvidia pushed a commit that referenced this pull request Jul 17, 2026
Signed-off-by: shanmugamr1992 <shanmugamr1992@gmail.com>
Signed-off-by: Arnav Kundu <arnavk@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants