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

build-essential compile_time attribute resolution order #81

Closed davidgiesberg closed 10 years ago

davidgiesberg commented 10 years ago

On a fresh box, I need to add this to an attributes file, otherwise the chef_gem 'zookeeper' directive will fail:

default['build-essential']['compile_time'] = true

My understanding is that this will resolve earlier in the chef run than the node.override line. Does that make sense to anyone else?

I'm not 100% clear what the Right Way to deal with this is, but this was my experience and fix that works for me.

jakedavis commented 10 years ago

Yep yep, you're right. Attribute files get evaluated before recipes, which are in turn evaluated in order. We should add this to the attributes file. I can take care of this in a couple hours or if you want to PR it we'd be happy to review. Thanks!

jakedavis commented 10 years ago

Fixed in #82.