fix: preserve batch wrapper exit status - #2838

Open

AdamRajfer wants to merge 1 commit into

NVIDIA-NeMo:mainfrom

AdamRajfer:arajfer/fix-batch-wrapper-shell

AdamRajfer commented Aug 28, 2026

Contributor

Summary

run the generated batch command with a non-login shell
preserve its exit status through server teardown
cover the submitted wrapper and clean lifecycle path

Tests

pytest -q tests/unit_tests/test_opensandbox_cleanup.py
pytest -q tests/unit_tests/ -m "not sandbox"
pre-commit run --all-files


          fix: preserve batch wrapper exit status

73fba88

Signed-off-by: Adam Rajfer <arajfer@nvidia.com>

copy-pr-bot Bot commented Aug 28, 2026

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet