search

sstarcher / docker-sensu

Dockerized Sensu - Client, Server, Api
MIT License
103 stars 72 forks source link

Consider switching to the sensu-install tool #51

Open sstarcher opened 7 years ago

ppawiggers commented 7 years ago

Simple example for installing plugins with sensu-install: https://github.com/ppawiggers/docker-sensu/blob/master/bin/install

sstarcher commented 7 years ago

The current install in docker-sensu supports a good bit more options, but ya the conversion should be fairly easy. I just need to drop some things in the script.

ppawiggers commented 7 years ago

Agree, that's why I did not submit a pull request for this feature, I currently just use this for myself.

guimaluf commented 7 years ago

Seems like load-checks doesn't work in current plugins install method. Cause load-checks requires a relative lib that isn't installed in the proper directory with gem build method.

/opt/sensu/embedded/bin/metrics-load.rb:37:in `require_relative': cannot load such file -- /opt/sensu/embedded/lib/sensu-plugins-load-checks/load-average.rb (LoadError)
sstarcher commented 7 years ago

require_relative should be fixed in that plugin. Plugins should not be using require_relative

sstarcher commented 7 years ago

This is a proper example of including a local library https://github.com/sensu-plugins/sensu-plugins-aws/blob/ffe7b24cf2b3f3f7ee3e92b08d40320377e2baa4/bin/check-beanstalk-elb-metric.rb#L30