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.

Sign up for GitHub

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

Jump to bottom

slurmdbd on separate host #2

Closed
uvNikita opened this issue Oct 31, 2017 · 2 comments
Closed

slurmdbd on separate host #2

uvNikita opened this issue Oct 31, 2017 · 2 comments
Labels
bug
Milestone
1.2

Comments

@uvNikita
Copy link
Contributor

It seems like in the current implementation of the module slurmdbd and slurmctld have to be on the same host. In particular, AccountingStorageHost is set to the current control host in the slurm.conf.

puppet-slurm/templates/slurm.conf.erb

Line 206 in c5782ed

AccountingStorageHost=<%= scope['slurm::controlmachine'] %>

It seems that just using db host instead should fix the problem (note that I didn't actually test that):

AccountingStorageHost=<%= scope['slurm::slurmdbd::dbdhost'] %>

I don't know much about slurm yet, so correct me please if I'm missing something.
@Falkor
Copy link
Contributor

Falkor commented Nov 7, 2017

Indeed, our current setup makes slurmdbd and slurmctld run on the same host.
We'll see for a new release of the module allowing for these changes, probably according to your suggestion.

@uvNikita
Copy link
Contributor Author

OK, thank you for the confirmation.

@uvNikita uvNikita changed the title slurmdbd on sparate host slurmdbd on separate host Nov 16, 2017
@Falkor Falkor added this to the 1.1 milestone Nov 22, 2017
@Falkor Falkor modified the milestones: 1.1, 1.2 Feb 26, 2018
@Falkor Falkor added the bug label Feb 26, 2018
Falkor pushed a commit that referenced this issue Feb 1, 2019
Possibility to set AccountingStorageHost
321a1db 
- Bugfix #2

Signed-off-by: Sebastien Varrette <Sebastien.Varrette@uni.lu>
@Falkor Falkor closed this as completed Feb 3, 2019
Falkor pushed a commit that referenced this issue Feb 3, 2019
Merge tag 'v1.2.0' into devel
4193635 
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
@uvNikita
Possibility to set AccountingStorageHost
eb4302f 
- Bugfix ULHPC#2

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
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
1.2
Development

No branches or pull requests
2 participants
@Falkor @uvNikita