fix: revert retries passthrough in core.py (breaks launcher unit test)

ChenhanYu · claude · ChenhanYu · commit f7bb9adc5faf · 2026-04-14T07:36:16.000-07:00
The launcher's SlurmConfig doesn't have retries field. Keep retries=0
hardcoded in core.py; nmm-sandbox's slurm.py handles retries separately.

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
Signed-off-by: Chenhan Yu &lt;chenhany@nvidia.com&gt;
diff --git a/tools/launcher/core.py b/tools/launcher/core.py
@@ -272,7 +272,7 @@ def build_slurm_executor(
         array=slurm_config.array,
         time=slurm_config.time,
         mem="0",
-        retries=getattr(slurm_config, "retries", 0),
+        retries=0,
         packager=packager,
         srun_args=slurm_config.srun_args,
     )

Original file line number	Diff line number	Diff line change
`@@ -272,7 +272,7 @@ def build_slurm_executor(`
`272`	`272`	`array=slurm_config.array,`
`273`	`273`	`time=slurm_config.time,`
`274`	`274`	`mem="0",`
`275`		`- retries=getattr(slurm_config, "retries", 0),`
	`275`	`+ retries=0,`
`276`	`276`	`packager=packager,`
`277`	`277`	`srun_args=slurm_config.srun_args,`
`278`	`278`	`)`