search

evertrue / zookeeper-cookbook

Chef cookbook for installing and managing Zookeeper.
https://supermarket.chef.io/cookbooks/zookeeper
Apache License 2.0
81 stars 117 forks source link

zookeeper doesn't reload/restart on config change #130

Closed davent closed 8 years ago

davent commented 9 years ago

When the config file specific with zookeeper_config is updated a reload/restart of zookeeper is not trigger in order to pick up the new changes.

jeffbyrnes commented 9 years ago

@davent that may be on-purpose; restarting ZK is not necessarily something that one wants to happen willy-nilly. I’ll defer to @jakedavis on that.

jakedavis commented 9 years ago

Yeah, IMO service management and config and installation should all be decoupled, and this is a pattern we try to use internally. That's a big reason we made sure to have a zookeeper_service resource, so one can easily trigger a restart as desired.

Edit: and as @jeffbyrnes mentions, restarting ZK is a thing you want to be very deliberate about.

jeffbyrnes commented 8 years ago

Closing due to lack of activity.