CoBrALab / qbatch

The Unlicense
27 stars 13 forks source link

logfiles without hand-made job names simply overwrite previous logs #146

Closed bcdarwin closed 6 years ago

bcdarwin commented 6 years ago
$ echo 'echo hello' | qbatch -
Submitted batch job 7650
$ cat logs/slurm-STDIN.out 
hello
$ echo 'echo goodbye' | qbatch -
Submitted batch job 7651
$ cat logs/slurm-STDIN.out 
goodbye
gdevenyi commented 6 years ago

Hrm, this seems like a slurm issue. Both SGE and PBS name their logfiles with a suffix of their jobID.

Will look into controlling naming of logfiles

gdevenyi commented 6 years ago

Looks like our logfile naming is overriding the inclusion of a jobid specs, will fix https://github.com/pipitone/qbatch/blob/master/bin/qbatch#L549

gdevenyi commented 6 years ago

Thanks for the good catch. We're not actively using SLURM yet, so I expect there to be some more tweaks to be done here.