puppetlabs / puppetlabs-ntp

Puppet module to manage the NTP service
http://forge.puppetlabs.com/puppetlabs/ntp
Apache License 2.0
145 stars 328 forks source link

minimum value for poll interval should be 3, not 4 #665

Closed johnwarburton closed 1 year ago

johnwarburton commented 1 year ago

Hi Team

Just migrating over to using the ntp forge module, and found this little nit

minpoll should be allowed to be 3, per the ntp documentation https://doc.ntp.org/documentation/4.2.6-series/confopt/#command-options, and 3 is even described as the lower range in https://github.com/puppetlabs/puppetlabs-ntp/blob/main/manifests/init.pp#L105

CLAassistant commented 1 year ago

CLA assistant check
All committers have signed the CLA.

johnwarburton commented 1 year ago

Thanks @chelnak Is there an ETA for the next release? Cheers

chelnak commented 1 year ago

I'm sure we can get something out asap!