docker-sensu icon indicating copy to clipboard operation
docker-sensu copied to clipboard

Consider switching to the sensu-install tool

Open sstarcher opened this issue 7 years ago • 6 comments

sstarcher avatar Jul 13 '17 12:07 sstarcher

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

ppawiggers avatar Jul 21 '17 15:07 ppawiggers

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.

sstarcher avatar Jul 21 '17 15:07 sstarcher

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

ppawiggers avatar Jul 21 '17 19:07 ppawiggers

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)

guimaluf avatar Nov 23 '17 11:11 guimaluf

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

sstarcher avatar Nov 24 '17 09:11 sstarcher

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

sstarcher avatar Nov 24 '17 09:11 sstarcher