Allow logconfig option in ntp.conf

[ericpaliotta]

🙍 Problem Statement

I have a use-case for which I would like to have more verbose ntp logging on some Ubuntu servers. This can be achieved using the logconfig =all option in the ntp.conf file. However, it seems like the ntp.conf.erb file in this repository does not have this option available via an attribute.

❔ Possible Solution

There is a simple solution to add a few lines to ntp.conf.erb which is something like this (in pseudo-code):

<% node['ntp']['logconfig_all'] -%>
logconfig =all
<% end -%>

I think this should work on most OSs (not sure though, I am not very familiar with NTP).

⤴️ Describe alternatives you've considered

I've considered using a bash resource to append this option manually to the ntp.conf. In addition, I tried to overwrite the config file using my own template. However, doing so triggers an ntp restart on each Chef run (this is not ideal for a number of reasons).

P.S. Please let me know if you need more information, or if I have missed a preexisting solution to this problem!