Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

Implement zookeeper_service #86

Closed
jakedavis opened this issue Jun 10, 2014 · 8 comments · Fixed by #175
Closed

Implement zookeeper_service #86

jakedavis opened this issue Jun 10, 2014 · 8 comments · Fixed by #175

Comments

@jakedavis
Copy link
Contributor

The Zookeeper docs recommend process supervision. As a baseline, we should have a resource which basically calls zkServer.sh start. But as a more resilient solution, we should probably have a Runit option, and possibly others depending on peoples' cup of tea.

@jeffbyrnes
Copy link
Contributor

What's the status of this, @jakedavis? Had some great momentum going!

@jakedavis
Copy link
Contributor Author

Ha, status is we have a service internally and maybe I should slurp it into this cookbook :) I think I did take a swing at a zookeeper_service resource, but it got messy quickly when I tried to be dynamic about it.

Thinking at the least we can add a node[:zookeeper][:service_style] attribute that defaults to runit, run a case on it in a service recipe and then just have Runit for now. The Sensu cookbook kind of does this (here), so maybe using that as inspiration might work.

@jeffbyrnes
Copy link
Contributor

@jakedavis sounds like a great path to tread to get this to a minimally working state. Thanks also for access to SimpleFinance/chef-exhibitor.

@jakedavis
Copy link
Contributor Author

Alright, took a swing at it in #91

@solarce
Copy link
Contributor

solarce commented Aug 21, 2014

Since we usually use upstart to manage services, here's a stab at some upstart supprt

@corbesero
Copy link

Is there some way to control what user zookeeper is run as using runit? If I use systemv, zookeeper runs as zookeeper. But with runit, it always seems to start up as root.

@jeffbyrnes
Copy link
Contributor

@corbesero that’d be a separate issue, which I’ve opened here: #166. This issue is meant to cover a refactor of how the service is set up; it should be refactored as its own custom resource.

@jeffbyrnes
Copy link
Contributor

I think this would best be served as part of a larger refactor of the LWRPs into Chef 12.5-style custom resources.

jeffbyrnes added a commit that referenced this issue Jun 17, 2016
Recipe → custom resource!

Closes #86

The runit_service template can be overridden if the operator wants to
supply their own runit service template
jeffbyrnes added a commit that referenced this issue Jun 20, 2016
Recipe → custom resource!

Closes #86

The runit_service template can be overridden if the operator wants to
supply their own runit service template
jeffbyrnes added a commit that referenced this issue Jun 21, 2016
Recipe → custom resource!

Closes #86

The runit_service template can be overridden if the operator wants to
supply their own runit service template
jeffbyrnes added a commit that referenced this issue Jun 21, 2016
Recipe → custom resource!

Closes #86

* The service template can be overridden if the operator wants to supply
their own service script template
jeffbyrnes added a commit that referenced this issue Jun 21, 2016
Recipe → custom resource!

Closes #86

* The service template can be overridden if the operator wants to supply
their own service script template
jeffbyrnes added a commit that referenced this issue Jun 21, 2016
Recipe → custom resource!

Closes #86

* The service template can be overridden if the operator wants to supply
their own service script template
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants