Skip to content

Commit

Permalink
Merge pull request #120 from mhakala/master
Browse files Browse the repository at this point in the history
some parameters deprecated on Slurm >=21.08
  • Loading branch information
VilleS1 authored Sep 10, 2021
2 parents b5f6687 + 94d5af8 commit 64d49d6
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
3 changes: 3 additions & 0 deletions defaults/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,9 @@ siteName: "io"
nodeBase: "{{ siteName }}"
nodeGpuBase: "gpu"

# this role has been written to support slurm >=18.08. Some conf parameters have been deprecated from newer Slurm versions so this is for backward compatibility
slurm_major_version: 18

# Have systemd restart slurm daemons if they fail to start
slurm_systemd_override_slurmd: True
slurm_systemd_override_slurmdbd: True
Expand Down
6 changes: 6 additions & 0 deletions templates/slurm.conf.j2
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,9 @@ MpiParams={{ slurm_mpi_params }}
SlurmctldPidFile={{ slurm_SlurmctldPidFile }}
SlurmdPidFile={{ slurm_SlurmdPidFile }}
ProctrackType={{ slurm_proctrack_type }}
{% if slurm_major_version|int < 21 %}
CacheGroups={{ slurm_CacheGroups }}
{% endif %}
FirstJobId={{ slurm_first_job_id }}
ReturnToService={{ slurm_return_to_service }}
MaxJobCount={{ slurm_max_job_count }}
Expand Down Expand Up @@ -70,7 +72,9 @@ SchedulerType={{ slurm_SchedulerType }}
SchedulerParameters={{ slurm_scheduler_parameters }}
SelectType={{ slurm_select_type }}
SelectTypeParameters={{ slurm_select_type_parameters }}
{% if slurm_major_version|int < 21 %}
FastSchedule={{ slurm_fast_schedule }}
{% endif %}
PriorityType={{ slurm_priority_type }}
{% if slurm_priority_type == "priority/multifactor" %}
PriorityFlags={{ slurm_priority_flags }}
Expand All @@ -97,7 +101,9 @@ AcctGatherEnergyType={{ slurm_AcctGatherEnergyType }}
AcctGatherNodeFreq={{ slurm_AcctGatherNodeFreq }}
AccountingStorageType={{ slurm_accounting_storage_type }}
AccountingStorageHost={{ slurm_accounting_storage_host }}
{% if slurm_major_version|int < 21 %}
AccountingStorageLoc={{ slurm_accounting_storage_loc }}
{% endif %}
AccountingStorageUser={{ slurm_accounting_storage_user }}
AccountingStorageEnforce={{ slurm_accounting_storage_enforce }}
{% if slurm_accounting_storage_tres is defined %}
Expand Down

0 comments on commit 64d49d6

Please sign in to comment.