search

saltstack-formulas / ntp-formula

http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
23 stars 158 forks source link

Updating formula so that changes to ntp.conf restart service #22

Closed markwherry9 closed 8 years ago

markwherry9 commented 8 years ago

Hi,

I found that a service restart was required after applying the formula to get NTP using the options specified in /etc/ntp.conf, so I added a watch on the configuration file to trigger a service restart.

Regards,

Mark

gravyboat commented 8 years ago

The watch should be on ntp_conf, as opposed to /etc/ntp.conf shouldn't it? Otherwise we don't support custom confs, and there's no actual declaration of an ID with the name /etc/ntp.conf.

markwherry9 commented 8 years ago

Hi Gravyboat,

Thanks for the feedback - now incorporated. In usage I found that ntp now correctly peers with the servers specifed in pillar data rather than package defaults. I'm fairly new to Saltstack so I hope I've got this right now.

Regards,

Mark

gravyboat commented 8 years ago

Good stuff, thanks @markwherry9!