search

geerlingguy / ansible-role-ntp

Ansible Role - NTP
https://galaxy.ansible.com/geerlingguy/ntp/
MIT License
318 stars 241 forks source link

Allow for multi-line ansible_managed values #123

Closed marknl closed 1 year ago

marknl commented 1 year ago

When using a multi-line ansible_managed, the current template will make chronyd not start since not all lines are commented. This will fix that.

bammbr commented 1 year ago

I second that. Similar changes for templates/ntp.conf.j2 were made four years ago. https://github.com/geerlingguy/ansible-role-ntp/commit/f5c9f54e8ca09a11a2778826857c3fcb150a470e

marknl commented 1 year ago

Hi @geerlingguy, Would you be so kind and merge this PR? Thanks in advance!

subbink commented 1 year ago

Is there anything I can do to get this merged?