Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compatibility with sensu-plugin 2.0.0 #71

Open
hwatts opened this issue Apr 28, 2017 · 4 comments
Open

Compatibility with sensu-plugin 2.0.0 #71

hwatts opened this issue Apr 28, 2017 · 4 comments

Comments

@hwatts
Copy link

hwatts commented Apr 28, 2017

I'm getting an error running checks with the latest sensu plugin version. It works if the plugin's downgraded to 1.4.5.

Are there any plans to support this?

/opt/sensu/embedded/bin/ruby /etc/sensu/plugins/elasticsearch/check-es-cluster-health.rb /opt/sensu/embedded/lib/ruby/site_ruby/2.4.0/rubygems/specification.rb:2288:in raise_if_conflicts': Unable to activate sensu-plugins-elasticsearch-1.1.3, because sensu-plugin-2.0.0 conflicts with sensu-plugin (~> 1.2) (Gem::ConflictError)

@majormoses
Copy link
Member

@hwatts thanks for reporting this, quite a few of the checks need to be updated to use 2.x @mattyjones any idea on when we are looking to start bumping the requirements on sensu-plugin?

@eheydrick
Copy link
Contributor

@hwatts I see you're executing the check outside the embedded ruby dir. If you execute it using the binstub in the embedded ruby (e.g. /opt/sensu/embedded/bin/check-es-cluster-health.rb) it'll load with sensu-plugin 1.4.5 and execute properly. sensu 0.29 includes both sensu-plugin 1.4.5 and 2.x so that should be all you have to do.

@eheydrick
Copy link
Contributor

@majormoses i think given that 0.29+ ships with 2.x we can start targeting it. We'll want to bump it in a major release since there are some potentially breaking changes. @cwjohnston any thoughts?

@majormoses
Copy link
Member

As a prerequisite to cutting from 1 to 2 I am going around to all the repos in 2 passes [1] to ensure ruby 2.4 support as it will be a nightmare if only some of the gems are bumped to 2.x and will leave people in very broken states where they can only monitor half (probably less tbh) of their infra.

[1] Adding testing for install time in ruby 2.4 (in progress, the gems listed do not in) and after that is complete will need to run through all repos and add support, testing, etc to be able to verify that it can work on both rubies 2.3 and 2.4 which will be a ton of work and may require breaking changes. After that the work for bumping sensu-plugin to 2.x can begin which will also be a breaking change. I see for sure one or two breaking changes/releases per plugin required to get here. This will not be quick and I don't yet have a timeline but you can check the progress on the super issue I created here: sensu-plugins/community#26 which should have links to the dependent work I outlined above and will have much of the work linked where possible. I honestly recommend to anyone using sensu to not upgrade at this time past 0.28, pin their gems with non pessimistic versions, and give us time to stabilize the plugins and extension repos before charging ahead. A monitoring server is not useful unless you can actually have it monitor all the things you need.

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

No branches or pull requests

4 participants