Skip to content

Add local executor resource limits to Pelle profile - #1170

Merged
mahesh-panchal merged 2 commits into
nf-core:masterfrom
mahesh-panchal:uppmax-local-executor-settings
Aug 31, 2026
Merged

Add local executor resource limits to Pelle profile#1170
mahesh-panchal merged 2 commits into
nf-core:masterfrom
mahesh-panchal:uppmax-local-executor-settings

Conversation

@mahesh-panchal

@mahesh-panchal mahesh-panchal commented Aug 27, 2026

Copy link
Copy Markdown
Member

Summary

  • Cap the local executor on the pelle profile to 8 CPUs / 128GB memory, so processes marked to run locally (short jobs that don't go through Slurm) don't fan out and overburden the shared Pelle login node.

Cap local jobs so they don't overburden the shared login node. Bianca
and Miarka login nodes are dedicated per-project/user, so they're left
uncapped.

Memory follows UPPMAX's own documented 128GiB per-user quota on Pelle
login nodes; the CPU cap is not yet confirmed against an official
limit.
@mahesh-panchal
mahesh-panchal marked this pull request as ready for review August 27, 2026 09:14
@mahesh-panchal
mahesh-panchal requested a review from pontus August 27, 2026 09:20
@mahesh-panchal
mahesh-panchal merged commit 876159b into nf-core:master Aug 31, 2026
157 of 161 checks passed
@mahesh-panchal
mahesh-panchal deleted the uppmax-local-executor-settings branch August 31, 2026 11:33
@pontus

pontus commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

I thought I had approved this, sorry for the delay.

I think this is valid, but am a bit unsure about the actual effects - for a process marked with the local executor, will it just cause the execution to blow up if it requests more than these limits? Is there any better way to deal with it in that case (I can't easily think of one).

@mahesh-panchal

Copy link
Copy Markdown
Member Author

No, it doesn't blow up. It just queues them up locally, and runs a new task when enough resources are available.

@pontus

pontus commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

But if the requests are above the limits, enough resources will never become available?

@pontus

pontus commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

So I assume there should be a

Caused by:
  Process requirement exceeds available CPUs -- req: 16; avail: 8

in those cases?

@mahesh-panchal

Copy link
Copy Markdown
Member Author

These jobs should also be few and far between. I don't think any nf-core pipeline explicitly sets local executor on a process. This should only be set by short running jobs which shouldn't be submitted to the queue.

Ah, I get the question now. Sorry, nextflow will error and say there's not enough resources, but no one should be requesting this on the login node. Those kind of jobs should be going to slurm.

@pontus

pontus commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Yes, I think it's acceptable to leave this to the user to deal with (they'd be the ones who'd be doing this anyway), just wanted to check if others thought there was anything smart that could be done.

@mahesh-panchal

Copy link
Copy Markdown
Member Author

If they're using this profile and this happens, it means they've likely explicitly set the local executor for a process, and they should know how update it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants