vmwaretools::ntp: use onlyif instead of refresh

voxpupuli / puppet-vmwaretools

Puppet module to manage VMware Operating System Specific Packages for VMware tools installation.

http://forge.puppetlabs.com/razorsedge/vmwaretools

Apache License 2.0

34 stars 44 forks source link

vmwaretools::ntp: use onlyif instead of refresh #33

Open igalic opened 9 years ago

igalic commented 9 years ago

it would be nice if vmwaretools::ntp would use a onlyif instead of refreshonly. to quote @ffrank: "Friends Don't Let Friends Use Refreshonly"

razorsedge commented 9 years ago

Unfortunately, the current approach is the only way to not have the vmware-guestd command execute on every Puppet run. It might be possible to convert to onlyif for the vmtoolsd command as it has a status check.