Skip to content
This repository has been archived by the owner on Feb 26, 2020. It is now read-only.

Consider switching to the sensu-install tool #51

Open
sstarcher opened this issue Jul 13, 2017 · 6 comments
Open

Consider switching to the sensu-install tool #51

sstarcher opened this issue Jul 13, 2017 · 6 comments

Comments

@sstarcher
Copy link
Owner

No description provided.

@ppawiggers
Copy link

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

@sstarcher
Copy link
Owner Author

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
Copy link

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

@guimaluf
Copy link

guimaluf commented Nov 23, 2017

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
Copy link
Owner Author

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

@sstarcher
Copy link
Owner Author

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants