IKIM-Essen / EMCP-config

IaC configuration of the Essen Medical Computing Platform (EMCP)
BSD 2-Clause "Simplified" License
0 stars 4 forks source link

Remove slurm controller from ntpclient play #118

Closed hfrSchmidt closed 1 year ago

hfrSchmidt commented 1 year ago

Otherwise the failure to enable and start the systemd unit timesyncd prevents the controller from being targeted in the subsequent plays.

enasca commented 1 year ago

This technically works, but it reads as if we don't want the slurm controller to be synchronised with our time server. Ideally we should communicate that we do want the slurm controller in sync, but there's nothing to do because it's containerised. I'll see if there's a convenient way for the ntpclient role to detect when it runs in a container and, if so, just do nothing.

enasca commented 1 year ago

Or perhaps it would be easier to simply assume that if the init system is not systemd, then the ntpclient role should just do nothing. Ansible stores this information in ansible_facts['service_mgr'].

hfrSchmidt commented 1 year ago

Good idea. Seems cleaner. I'll push the changes sometime today.