Hardcode SlurmdSpoolDir to the default /var/spool/slurmd

openhpc_slurmd_spool_dir was not used in slurm.conf

Author: elelaysh

README.md

@@ -149,8 +149,6 @@ that this is *not the same* as the Ansible `omit` [special variable](https://doc
 
 `openhpc_state_save_location`: Optional. Absolute path for Slurm controller state (`slurm.conf` parameter [StateSaveLocation](https://slurm.schedmd.com/slurm.conf.html#OPT_StateSaveLocation))
 
-`openhpc_slurmd_spool_dir`: Optional. Absolute path for slurmd state (`slurm.conf` parameter [SlurmdSpoolDir](https://slurm.schedmd.com/slurm.conf.html#OPT_SlurmdSpoolDir))
-
 `openhpc_slurm_conf_template`: Optional. Path of Jinja template for `slurm.conf` configuration file. Default is `slurm.conf.j2` template in role. **NB:** The required templating is complex, if just setting specific parameters use `openhpc_config` intead.
 
 `openhpc_slurm_conf_path`: Optional. Path to template `slurm.conf` configuration file to. Default `/etc/slurm/slurm.conf`

defaults/main.yml

@@ -18,7 +18,6 @@ openhpc_default_config:
   SlurmctldHost: "{{ openhpc_slurm_control_host }}{% if openhpc_slurm_control_host_address is defined %}({{ openhpc_slurm_control_host_address }}){% endif %}"
   ProctrackType: proctrack/cgroup
   TaskPlugin: task/cgroup,task/affinity
-  SlurmdSpoolDir: /var/spool/slurm # NB: not OpenHPC default!
   SlurmUser: slurm
   StateSaveLocation: "{{ openhpc_state_save_location }}"
   SlurmctldTimeout: 300
@@ -68,7 +67,6 @@ openhpc_mpi_template: mpi.conf.j2
 openhpc_mpi_config: {}
 
 openhpc_state_save_location: /var/spool/slurmctld
-openhpc_slurmd_spool_dir: /var/spool/slurm
 openhpc_slurm_conf_path: /etc/slurm/slurm.conf
 openhpc_slurm_conf_template: slurm.conf.j2
 

tasks/runtime.yml

@@ -21,7 +21,7 @@
       enable: control
     - path: "{{ openhpc_slurm_conf_path | dirname }}"
       enable: control
-    - path: "{{ openhpc_slurmd_spool_dir }}" # SlurmdSpoolDir
+    - path: "/var/spool/slurmd" # SLURM's default for SlurmdSpoolDir
       enable: batch
   when: "openhpc_enable[item.enable] | default(false) | bool"