bbaugher / apache_zookeeper

Chef cookbook for Apache Zookeeper
MIT License
9 stars 22 forks source link

Issue/dcode/25 #26

Closed dcode closed 9 years ago

dcode commented 9 years ago

I decided to go ahead and do the pull request for what I think makes sense. Ultimately, I want to use systemd to load the process, but I doctored up the init script to work around the lack of user shell. I tested in CentOS, but read that the /etc/init.d/functions file exists on at least Ubuntu too, so hopefully that's works?

It shouldn't be too hard to generate init files using the SysV init style (like here), systemd, and even upstart. I'm willing to rewrite the SysV init script and the systemd unit file, but I'll do that in another patch.

bbaugher commented 9 years ago

Yes this looks good. I was thinking about moving to upstart or runit. Thanks for the pull