nextflow icon indicating copy to clipboard operation
nextflow copied to clipboard

Add account config option for grid executors

Open pditommaso opened this issue 3 months ago • 6 comments

This PR adds the ability to define a Slurm account in the nextflow config. For example:

executor {
  account = 'some-project-xyz'
}

Solve https://github.com/nextflow-io/nextflow/issues/4921

Updated to use executor instead of slurm on 7th May

pditommaso avatar May 06 '24 20:05 pditommaso

Deploy Preview for nextflow-docs-staging canceled.

Name Link
Latest commit 08101ec4ea2b115799a2fd44a6890a061a3a5b64
Latest deploy log https://app.netlify.com/sites/nextflow-docs-staging/deploys/663b3739cd44440008f37a68

netlify[bot] avatar May 06 '24 20:05 netlify[bot]

In this case I recommend also moving executor.perCpuMemAllocation to slurm.perCpuMemAllocation.

bentsherman avatar May 06 '24 20:05 bentsherman

I would suggest exposing this option as a generic executor one, executor.account, rather than creating the new slurm config scope.
Indeed, other executors implement it, e.g. PBS has -P <account/project>.

marcodelapierre avatar May 07 '24 05:05 marcodelapierre

Hi. I was going to suggest the same thing as @marcodelapierre . LSF has the -G user_group: https://www.ibm.com/docs/en/spectrum-lsf/10.1.0?topic=o-g-1

muffato avatar May 07 '24 08:05 muffato

Fair enough

pditommaso avatar May 07 '24 09:05 pditommaso

@bentsherman @marcodelapierre please have a look and let's merge it

pditommaso avatar May 07 '24 20:05 pditommaso

Thanks both. I've reverted 9486d68 because it's better to keep explicit from the name is coming from

pditommaso avatar May 08 '24 08:05 pditommaso