hectcastro / chef-collectd

A Chef cookbook to install collectd.
Apache License 2.0
23 stars 65 forks source link

Update python/curl_json dependencies and explicitly enable plugins. #34

Closed austburn closed 9 years ago

austburn commented 9 years ago

While using this cookbook, I noticed that the curl_json and python plugins did not work "out of the box." I used the collectd source to diagnose the problem and figure out the fix.

It appears to me that many of the plugins for collectd don't come via default installation. I have not investigated more than curl_json or python, but I they are some examples of plugins that are not included with the base installation, i.e. configure && make all install.

Essentially, this pull request takes the keys from the plugins hash and explicitly enable plugins in the configuration stage. I had to add curl_json and python dependencies to enable these plugins.

I'm willing to investigate the other plugins that are not being included by default and if they need dependencies resolved, but I wanted to get the ball rolling on this issue.

Feedback welcome!

calebreach commented 9 years ago

Looks good to me.