bpipe icon indicating copy to clipboard operation
bpipe copied to clipboard

Support for generic job submission options

Open ssadedin opened this issue 8 years ago • 8 comments

Some executors (eg: Slurm) don't have support for options to allow an arbitrary command line parameter to be added to the job submission. It would be good to support this with one general config option rather than a specific one for each executor.

See discussion on google group.

ssadedin avatar Jan 04 '17 15:01 ssadedin

Tested it with slurm 16.05.5 and am getting an error:

Test mode: Abort due to Test Mode!

Branch aFile would execute: touch aFile.test_module

                using Slurm Job [Command Id: null ] with config [max_per_command_threads:16, executor:slurm, walltime:00:01:00, procs:1, queue:ikmb_a, memory:1, custom_submit_options:--qos=ikmb_a, name:test]

--> looks OK!

Execute mode:

ERROR: stage test_module failed: Job runner bpipe.executor.SlurmCommandExecutor failed to return a job id for job 2 (test_module) despite reporting success exit code for command:

bash /ifs/data/nfs_share/ikmb_repository/software/bpipe/devel/bin/../bin/bpipe-slurm.sh start

Raw output was:[ ]

less .bpipe/commandtmp/2/job.slurm

#!/bin/bash #SBATCH --job-name=test_module

#SBATCH --mem=1024 #SBATCH --time=00:01:00 #SBATCH --ntasks=1 #SBATCH --nodes=1 #SBATCH -p ikmb_a

set -o errexit

(touch aFile.test_module) > .bpipe/commandtmp/2/2.out 2> .bpipe/commandtmp/2/2.err

marchoeppner avatar Jan 10 '17 09:01 marchoeppner

Maybe to clarify - the Bpipe test mode shows that the optional parameter "--qos=ikmb_a" is seen by the tool, but the slurm submit script does not include this option. Somewhere between reading all parameters and writing the execution script, the new option is lost.

marchoeppner avatar Jan 12 '17 08:01 marchoeppner

Can you look in the bpipe log file (.bpipe/bpipe.log) and search for the sbatch command? hopefully it should be logged in there and we can see exactly what is being used.

Also, could you just confirm - you're building from the branch I created, not master, correct?

ssadedin avatar Jan 12 '17 14:01 ssadedin

Hi,

so I redid the whole thing and arrived add an compilation error:

git clone https://github.com/ssadedin/bpipe.git devel

git fetch

git checkout -b custom_submit_opts origin/custom_submit_opts Branch custom_submit_opts set up to track remote branch custom_submit_opts from origin. Switched to a new branch 'custom_submit_opts'

git branch

  • custom_submit_opts master

/gradlew dist

================================================================================ No Gridgain library found. Gridgain support disabled in this build.

:compileJava UP-TO-DATE :compileGroovy startup failed: /ifs/data/nfs_share/ikmb_repository/software/bpipe/devel/src/main/groovy/bpipe/CommandManager.groovy: 327: unexpected token: << @ line 327, column 1. <<<<<<< HEAD ^

1 error

:compileGroovy FAILED

marchoeppner avatar Jan 16 '17 07:01 marchoeppner

Sorry about that ... don't quite know how it happened but hopefully fixed now. Can you try again?

ssadedin avatar Jan 17 '17 13:01 ssadedin

Hi, seems to work now, thanks.

marchoeppner avatar Jan 18 '17 10:01 marchoeppner

Hi, anything else that needs testing here? Or will this be merged into master now? /M

marchoeppner avatar Mar 20 '17 09:03 marchoeppner

Should be merged in now.

On Mar 20, 2017 8:05 PM, "marchoeppner" [email protected] wrote:

Hi, anything else that needs testing here? Or will this be merged into master now? /M

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ssadedin/bpipe/issues/193#issuecomment-287705468, or mute the thread https://github.com/notifications/unsubscribe-auth/AAIedLYuR-Y1lJ7MTjHIUZpGYpTgB2ZGks5rnkFGgaJpZM4LauA- .

ssadedin avatar Mar 20 '17 11:03 ssadedin