Issue #3323 has been updated by James Turnbull. Status changed from Requires CLA to be signed to In Topic Branch Pending Merge
Hi Chuck - hope you don't mind I updated and extended your code and submitted it as a pull request. It now supports reload and graceful and I extended the test coverage a little also. ---------------------------------------- Feature #3323: Create hasreload similar to hasrestart https://projects.puppetlabs.com/issues/3323 Author: Chuck McIntyre Status: In Topic Branch Pending Merge Priority: Normal Assignee: Chuck McIntyre Category: service Target version: Affected Puppet version: 0.24.8 Keywords: hasreload, service Branch: https://github.com/jamtur01/puppet/tree/tickets/master/3323 Many init scripts allow for a "reload" option in addition to a restart and a stop/start. It would be useful if this could be defined for a given service generically, rather than having to resort to an Exec resource or other workaround for cases where we want to reload instead of restart (an example could be made using apache or bind configuration changes). I am attaching an example diff (note this is not a patch, nor has it been tested) to describe in more detail how this might be accomplished. I am willing to submit a patch for this if the feature is accepted. I would like some guidance as to whether the existing unit test modifications are sufficient or if it is necessary to test that (at least) reload and restart get called appropriately (not simply that stop/start get called when not specified). -- 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.
