Skip to content

Commit

Permalink
Merge branch 'RESTAPI-1185-fix-sbatch-export-parameter' into 'master'
Browse files Browse the repository at this point in the history
added ALL to --export option on SLURM

See merge request firecrest/firecrest!317
  • Loading branch information
Juan Pablo Dorsch committed Aug 27, 2024
2 parents e00c3b7 + fd89a5c commit ed3c38d
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/common/schedulers/slurm.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,8 @@ def submit(self, submission_spec):
text += f"{k}={v},"
text = text[:-1]
job_env = text
cmd.append(f"--export='{job_env}'")
# if --export option is used, the ALL value has to be set as well
cmd.append(f"--export='ALL,{job_env}'")

cmd += [f"--chdir='{submission_spec.job_dir}'"]
cmd += self._opts
Expand Down

0 comments on commit ed3c38d

Please sign in to comment.