Need to make sure apt/yum cookbooks are loaded when dd-handler is run

[alq666]

Can move this to default and include it in both dd-agent and dd-handler recipes.

[miketheman]

Can you elaborate as to why apt/yum are needed for dd-handler?

[alq666]

ruby-dev needed to compile json iirc.

On Aug 9, 2012, at 8:26 AM, Mike Fiedler notifications@github.com wrote:

Can you elaborate as to why apt/yum are needed for dd-handler?

—
Reply to this email directly or view it on GitHub.

[miketheman]

Ok, it seems like it's a requirement for yajl-ruby before it's a requirement for json, but also a requirement there.

I wonder id any system installing a modern version of chef won't already have this?

[alq666]

I interpret https://github.com/DataDog/chef-datadog/blob/master/metadata.rb#L9 as a solution to this bug.

[miketheman]

I support closure of this bug, but think it might be for the wrong reason. Adding a dependency to the metadata doesn't ensure that any actions/resources are activated unless an include_recipe statement is run, such as is used here.

Also, since the handlers are being installed at compile time, I suspect we should probably remove these lines, as json, yajl-json are requirements of chef itself, and would have to have been resolved prior to running chef itself.