Issue #6711 has been updated by Nigel Kersten. Category changed from exec to documentation Assignee set to Nick Fagerlund Priority changed from Normal to Low
We should just document this for cases where there is a separate refresh command. ---------------------------------------- Bug #6711: exec type refresh cmd should do tries and log https://projects.puppetlabs.com/issues/6711 Author: Matt Robinson Status: Accepted Priority: Low Assignee: Nick Fagerlund Category: documentation Target version: Affected Puppet version: development Keywords: Branch: # Run the command, or optionally run a separately-specified command. def refresh if self.check_all_attributes(true) if cmd = self[:refresh] provider.run(cmd) else self.property(:returns).sync end end end If a refresh parameter is given to an exec type with tries set exec { '/some/cmd' : refresh => '/some/othercmd', tries => 10, } tries will only apply to the /some/cmd, and not to the /some/othercmd when the exec resource is triggered by a refresh. This is probably unintuitive and should be fixed or at least documented. Also, by having different code paths for running the command regularly and through a refresh some logging gets lost when refreshing. -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
