search

etsy / nagios-herald

Add context to Nagios alerts
https://codeascraft.com/2014/06/06/introducing-nagios-herald/
MIT License
321 stars 32 forks source link

Removed chef helpers from ganglia_graph.rb to remove chef dependancy #22

Closed Raizex closed 10 years ago

Raizex commented 10 years ago

Commented out chef helpers as per kargig's suggestion in https://github.com/etsy/nagios-herald/issues/19

Nothing else in the code base seems to make use of them at the moment, and this seems to have solved the chef dependancy issue.

Let me know if this seems to break anything on your end.

RyanFrantz commented 10 years ago

Unfortunately, I don't think this solution works well because we do require Chef to look up the Ganglia cluster name. 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.