Comments (10)
@eyalzek hmm… seems some refactor of the actions for zookeeper_service
is in order. It should provide a mechanism to pass through any notifications to its inner resources (as it is an abstraction of your chosen service provider).
The technical reason for the error is because service[zookeeper]
doesn’t exist in the scope of your recipe; it’s inside the inner scope of the zookeeper_service[zookeeper]
resource you have.
For now, you can get things up & running by omitting the notifies
, and manually restart ZK.
I’d welcome some help with this, as I likely won’t have much time to devote to this for a while.
from zookeeper-cookbook.
I took a look at it and as you said, it seems like it requires a refactor more than a minor 2-line change. I might be able to help if you point me in the right direction.
from zookeeper-cookbook.
@eyalzek thanks! If you check out resources/service.rb
, you’ll see that it has two actions:
:create
:delete
Taking a cue from poise_service
, I think it’s better if it had a set of actions that correspond 1:1 with the underlying service resources:
actions :enable, :disable, :start, :stop, :restart, :reload
In fact, now that I think about it, it might be sensible to just use poise_service
, since that solves this for us…
from zookeeper-cookbook.
I took a crack at using poise-service to create the zookeeper_service resource. So far I've only implemented it with upstart(and poorly at that) and wanted to get your thoughts before putting much more time into this.
from zookeeper-cookbook.
@cullenmcdermott oh nice! That looks pretty good so far. If you want to open a pull request with that work, we can try to get it fully fleshed out together.
Two notes:
- Please leave the version number alone in the PR (I’ll handle that on
master
when the code merges in & I can properly release it) - Rather than comment out the content of
resources/service.rb
, just delete the file
from zookeeper-cookbook.
Hey sorry, its been a crazy week so far. I'll try to get a PR submitted tonight or tomorrow. Something with what I had committed so far wasn't working so I refactored it a bit and have it working better now.
from zookeeper-cookbook.
@jeffbyrnes I just created #191
from zookeeper-cookbook.
Hi, any chance this will be merged?
We have a large cluster and need an ability to restart the service on config change when new servers added/removed.
from zookeeper-cookbook.
@heaven I wager you mean #191?
tl;dr, probably not anytime soon.
I’ve had very few cycles to put towards this, especially since I no longer use ZK (job change, no need for it at the new gig).
However, I caution: this would not handle restarts gracefully (i.e., rolling restart), and thus the potential for data loss exists.
In the past, I’ve solved that with Exhibitor, but that project has seen some ups & downs since it’s departure from Netflix, so I recommend taking a hard look at it. We’ve a cookbook that used to play nicely with this one for managing all of that, but again, maintenance has been minimal lately.
Moving forward, the ZooKeeper team is focused on providing dynamic configuration, including resizing of the cluster. It’s available starting in v3.5.0, and here are the latest docs for v3.5.3-beta.
I realize using software that’s yet to be marked stable may not be an option, and this cookbook doesn’t even properly support v3.5.x, so that may not be optimal, but perhaps you can pick up #199 and finish up the v3.5.x support, and use that.
Sorry I’m not able to offer more support, and since I’m not the owner of this, I can’t exactly hand off maintenance (though I do still possess admin, so I can grant contributor if you want to step in).
from zookeeper-cookbook.
@heaven so I did manage to squeeze in some work and get the zookeeper-env.sh
fixes (and some others) done & released as v10.0.0.
Fixing this is a much larger refactor, where the zookeeper_service
resource is rewritten to be a more direct wrapper around its internals. I would very much welcome any assistance with that; #191 is a good starting point.
And as I said, it might be good to get #199 finished up.
from zookeeper-cookbook.
Related Issues (20)
- Upstart services fail to start zookeeper HOT 8
- Question about your pinning strategy HOT 2
- link logic for ZK not working. HOT 1
- No way to set Java Opts? HOT 2
- cookbook dependancies in the metadata HOT 4
- Support "package" install style HOT 4
- Make the defaults for install_dir consistent across resources HOT 1
- Checksum has not changed with 3.4.9 update HOT 1
- Any feedback with ZK 3.5.2 dynamic configuration ? HOT 3
- Zookeeper sysv script
- How to run ZooKeeper as a service HOT 1
- Support Ubuntu 16.04 HOT 3
- support direct link to tar for download path HOT 2
- zookeeper-3.5.3-beta artifact breaks due to (lack of) gzip compression HOT 3
- 2.6.0 version error using chef-client 12 HOT 2
- MYID HOT 4
- Passing java options with special characters has unexpected output HOT 6
- Default install fails with failure to download Zookeeper 3.4.12 HOT 2
- Inconsistency in username properties across resources HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from zookeeper-cookbook.