galaxyproject / ansible-slurm

Ansible role for installing and managing the Slurm Workload Manager
86 stars 47 forks source link

Change Service restart order #17

Closed roumano closed 1 year ago

roumano commented 3 years ago

When restart multiple service , should restart in this order :

Fix #16

natefoo commented 1 year ago

Sorry for the delay in reviewing. The slurmdbd.conf handling was fixed in #19 and PIDFile stuff is no longer relevant as of #30, so I think it's just the handler reordering that is still applicable - I can revert the other changes and update your PR to get it merged.

roumano commented 1 year ago

Yes, it's will perfect like this ( just the handler reordering )