Issue #6711 has been updated by Matt Robinson.
Hey Nigel, I assigned this to you with needs design decision since currently refresh commands don't get run the number of times tries specifies, and perhaps this is desired behavior. If the ticket doesn't make sense let me know. ---------------------------------------- Bug #6711: exec type refresh cmd should do tries and log https://projects.puppetlabs.com/issues/6711 Author: Matt Robinson Status: Needs Decision Priority: Normal Assignee: Nigel Kersten Category: 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.
