Ensure config_dir exists

evertrue / zookeeper-cookbook

Chef cookbook for installing and managing Zookeeper.

https://supermarket.chef.io/cookbooks/zookeeper

Apache License 2.0

81 stars 119 forks source link

Ensure config_dir exists #158

Closed neil-greenwood closed 8 years ago

neil-greenwood commented 8 years ago

Fixes #157

jeffbyrnes commented 8 years ago

@neil-greenwood thanks! I’d suggest that the recursive property also be used, to ensure that any path, no matter how arcane, could be specified for the config_dir

neil-greenwood commented 8 years ago

I've found that this change isn't sufficient. The cookbook creates the config file in the correct place, but several of the ZK commands (e.g. zkServer.sh status) are still using the old location.

I'm still working on a complete fix.

jeffbyrnes commented 8 years ago

@neil-greenwood any chance you’ve gotten some more done on this?