Issue #3723 has been updated by Nigel Kersten.
I'm thinking it's more obvious to make it "tries" (defaultto 1) rather than "retries". Wondering what people would expect the timeout parameter to do in this case though... Should it apply to the whole run of this resource (including all tries) or should it apply to each "try" ? ---------------------------------------- Feature #3723: Retries and sleep options for Exec http://projects.puppetlabs.com/issues/3723 Author: Nigel Kersten Status: Accepted Priority: Normal Assigned to: Nigel Kersten Category: exec Target version: Affected version: 0.25.4 Keywords: Branch: We still find there are things around we need to use an Exec for, and sometimes these have transient failures due to NFS dependencies, etc etc. It would be quite useful if the Exec type supported the ability to retry these sorts of situations, ie <pre> exec { "some_transient_failing_command": retries => 3, sleep => 15, } </pre> with the idea being that if the exec returned an unacceptable exit code, it would retry "retries" times and sleep "sleep" seconds between each retry. People would need to be careful about ensuring their sleep time didn't exceed the exec timeout, but that seems reasonable enough. I'm happy to do this if it's accepted as an upstream feature. -- 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.
