From 29110d2d544bcc909d8d79a1b1a69ae9f8cebac1 Mon Sep 17 00:00:00 2001
From: Rodrigo Arias Mallo <rodrigo.arias@bsc.es>
Date: Thu, 18 Jul 2024 11:19:03 +0200
Subject: [PATCH] Allow other jobs to run in unused cores

The current select mechanism was using the memory too as a consumable
resource, which by default only sets 1 MiB per node. As each job already
requests 1 MiB, it prevents other jobs from running.

As we are not really concerned with memory usage, we only use the unused
cores in the select criteria.
---
 m/module/slurm-client.nix | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/m/module/slurm-client.nix b/m/module/slurm-client.nix
index 22ffae6..e81577e 100644
--- a/m/module/slurm-client.nix
+++ b/m/module/slurm-client.nix
@@ -83,6 +83,14 @@ in {
 
       # Reduce port range so we can allow only this range in the firewall
       SrunPortRange=60000-61000
+
+      # Use cores as consumable resources. In SLURM terms, a core may have
+      # multiple hardware threads (or CPUs).
+      SelectType=select/cons_tres
+
+      # Ignore memory constraints and only use unused cores to share a node with
+      # other jobs.
+      SelectTypeParameters=CR_CORE
     '';
   };