add support for using Slurm as backend for --job#2642
add support for using Slurm as backend for --job#2642akesandgren merged 12 commits intoeasybuilders:developfrom
Conversation
houndci-bot
left a comment
There was a problem hiding this comment.
Some files could not be reviewed due to errors:
Traceback (most recent call last):
Traceback (most recent call last):
File "/home/linters/.local/bin/flake8", line 7, in
from flake8.main.cli import main
ModuleNotFoundError: No module named 'flake8'
houndci-bot
left a comment
There was a problem hiding this comment.
Some files could not be reviewed due to errors:
Traceback (most recent call last):
Traceback (most recent call last):
File "/home/linters/.local/bin/flake8", line 7, in
from flake8.main.cli import main
ModuleNotFoundError: No module named 'flake8'
|
@akesandgren Via The default depends on the job backend (to retain backward compatibility): |
akesandgren
left a comment
There was a problem hiding this comment.
LGTM, should we go ahead and merge this now or do you think there is anything more to do?
|
@akesandgren I hope to add some tests for the Any chance that you can give this a spin yourself to see if |
|
create_job in parallelbuild sets up easybuild_vars to contain only |
|
@akesandgren That's orthogonal to this PR I think, see the discussion in #2632 |
|
There is something wrong. SlurmJob was passing the wrong parameter to pyslurm. It expects 'time_limit' and not 'time' and it expects an integer in minutes. |
And the value is an integer of minutes.
pyslurm expects time_limit, not 'time'.
|
@akesandgren I've just pushed support for using I'm wondering this is a better way forward: doesn't require having |
|
There needs to be a config option for choosing what to put in "#!/bin/sh" of the submit script. We require /bin/bash for things to work. But it seems that is done by sbatch itself... Not sure that is the problem i ran into now or not though. Nah, that wasn't the poroblem. Ignore this comment. |
|
With my latest PR to this one I can get stuff running. Will try something local later this evening. |
|
@boegel Anything else you plan on doing with this? |
|
@akesandgren @migueldiascosta is testing this too, doesn't hurt to get some more feedback on it. But it's good to go imho. |
|
@akesandgren docs update @ easybuilders/easybuild#467 |
With this in place, you configure EasyBuild to submit jobs to a SLURM cluster (using
--job-backend=PySlurm), as long as a sufficiently recent version ofPySlurmis installed (https://pypi.org/project/pyslurm/).