Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add SchedulerParameters to slurm.conf.erb template #13

Closed
stevca9 opened this issue Jul 11, 2018 · 1 comment
Closed

Add SchedulerParameters to slurm.conf.erb template #13

stevca9 opened this issue Jul 11, 2018 · 1 comment
Milestone

Comments

@stevca9
Copy link

stevca9 commented Jul 11, 2018

Could you please add support for the SchedulerParameters options in slurm.conf?

https://slurm.schedmd.com/slurm.conf.html

We are currently trying to implement some of the backfill features of the SLURM scheduler and would like to have the option to configure this through the module. According to the documentation the SchedulerParameters option can be defined as a comma separated list, so if I could define these parameters as an array of strings in the slurm class, then hopefully each of these parameters wouldn't need its own definition, and there'd only need to be one definition for the SchedulerParameters option.

Thank you!

@Falkor Falkor added this to the 1.2 milestone Jul 11, 2018
Falkor pushed a commit that referenced this issue Feb 2, 2019
- Bugfix #13

Signed-off-by: Sebastien Varrette <Sebastien.Varrette@uni.lu>
@Falkor
Copy link
Contributor

Falkor commented Feb 2, 2019

Example of hiera setting (multiline YAML with |) (used in tests/vagrant/puppet/hieradata/custom.yaml):

slurm::schedulerparameters:
  - 'sbatch_wait_nodes'
  - 'spec_cores_first'
  - 'sched_min_interval=12'

Which results in the following:

$> vagrant provision --provision-with puppet slurm-master
[...]
==> slurm-master: Info: Applying configuration version '1549121840'
==> slurm-master: Notice: /Stage[main]/Slurm::Config/File[slurm.conf]/content:
==> slurm-master: --- /etc/slurm/slurm.conf	2019-02-02 15:36:12.338590512 +0000
==> slurm-master: +++ /tmp/puppet-file20190202-3981-1k941sj	2019-02-02 15:37:27.881672774 +0000
==> slurm-master: @@ -178,7 +178,7 @@
==> slurm-master:  FastSchedule=1
==> slurm-master:  #SchedulerTimeSlice=30
==> slurm-master:  SchedulerType=sched/backfill
==> slurm-master: -#SchedulerParameters=
==> slurm-master: +SchedulerParameters=sbatch_wait_nodes,spec_cores_first,sched_min_interval=12
==> slurm-master:
==> slurm-master:  # Plugin used to identify which jobs can be preempted in order to start a pending job.
==> slurm-master:  PreemptType=preempt/qos
==> slurm-master:
[...]

@Falkor Falkor closed this as completed Feb 2, 2019
Falkor pushed a commit that referenced this issue Feb 3, 2019
TRES, scheduler parameters and custom content management

- Bugfix #2: slurmdbd on separate host bug
- Bugfix #7: slurm-login nodes without daemons
- Bugfix #12: slurm-17.11.3 install fails bug
- Bugfix #13: Add SchedulerParameters to slurm.conf.erb template
- Bugfix #14: Prevent improper use of SchedMD resources
- Bugfix #16: Allow for TRESBillingWeights enhancement
- Bugfix #17: Consolidate vagrant setup for module validation on a virtual cluster enhancement
- Bugfix #18: Allow for custom content enhancement
uvNikita pushed a commit to uvNikita/puppet-slurm that referenced this issue Apr 5, 2019
- Bugfix ULHPC#13

Signed-off-by: Sebastien Varrette <Sebastien.Varrette@uni.lu>
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

No branches or pull requests

2 participants