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

check for jar file instead of tarball #113

Closed pwyliu closed 9 years ago

pwyliu commented 9 years ago

I think I found a little bug. zk_installed always returns false because the tarball is not in install_dir, it's in the file cache. This makes the cookbook always want to extract and chown on every Chef run. I changed the check so it looks for the jar file instead.

jeffbyrnes commented 9 years ago

@pwyliu nice! Thanks for the fix.

jeffbyrnes commented 9 years ago

@jakedavis heads-up on this guy.

jakedavis commented 9 years ago

Oh nice this was buried in my 100 Chrome tabs :) word!