-
Notifications
You must be signed in to change notification settings - Fork 32
Only works with chef #19
Comments
Also, after installing chef (gem install chef), it doesnt work because of mismatched gem versions
|
Hmm, this is a vestige of some internal bits I'd mostly culled. I can remove the Chef dependency and replace it with something more generic that others can build on. Also, if you have any code you'd like to submit that does that, I'd love to see it. |
I guess one can comment out 'unneeded' helpers in lib/nagios-herald/helpers.rb, right ? |
Per #22, I'll modify this helper to be smarter about requiring Chef support and fall back to a simpler mechanism to look up the cluster name. |
@RyanFrantz I ran into similar issues whereby I did not install the chef gem (that requires ruby 2.0 btw which I did not want to muck on) - one of the commit in #36 fixes the import issue, but not the underlying mechanism of the ganglia_graph helper. I also have not removed the dependency in the gemspec - I am not sure yet what would be the best approach to manage that gem since it could be an optional gem based on every user situation. |
From what I understand (and I don't understand ruby ;)) this currently kinda depends on chef, as https://github.com/etsy/nagios-herald/blob/master/lib/nagios-herald/helpers/ganglia_graph.rb#L1 includes some chef code.
This should be clarified in the documentation or changed.
Without chef being installed it results in the following:
The text was updated successfully, but these errors were encountered: