search

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

Updates resource/config.rb to correctly set file owners and groups #209

Closed melissafritcher closed 6 years ago

melissafritcher commented 6 years ago

Signed-off-by: Melissa Fritcher mfritcher@us.ibm.com

The file and owner aren't being set correctly in the config resource. I tracked the issue down to https://docs.chef.io/dsl_custom_resource.html See new_resource.property section.

jeffbyrnes commented 6 years ago

@melissafritcher thanks for catching this! I ended up making similar changes due to a deprecation in Chef 13.2. I’ll incorporate your test adjustments as well, so we can catch this in the future if it happens again.