thias/puppet-bind — pull request #19 by asedge (2013-10-31 15:34:25+00:00)
Title: Adding managed-keys-directory option as well as giving module user ability to use service restart rather than service reload (at their own risk of course).

1st commit: In 'top' output, named process consuming 100% (or more in my SMP environment). Adding this option to the configuration fixes that issue, may be related to using views. Fixed ChangeLog as well, you gave me far more credit than I deserved.

2nd commit: The option was not being realized by named with a simple reload. Restarting named manually isn't practical in my environment where I don't personally administrate the servers. Would like to get this merged but recall seeing a discussion in the past where you made a valid point about not causing named to fail in case of misconfiguration. I made it optional and defaults to the behavior you defined: service $servicename reload.

I did this in 2 commits so it would be easy for you to cherry-pick, if you choose. Though hopefully you see that some users may want to assume all the risk in the case of a service restart. Thanks!