Ben Abrams
Ben Abrams
@eheydrick our dependency on 1.x is quite old, I am tempted to say we should try to bump it all the way to 5.x if possible if not we should...
@eheydrick Understood and hope we can maintain support for all 3, however we might want to drop 1.x if it causes issues as its now technically EOL. https://www.elastic.co/support/eol ``` Elasticsearch...
@kaushiksriram100 would you be willing to try bumping the dependencies and testing?
More than happy to help. ` s.add_runtime_dependency 'elasticsearch', ['>= 2.0.0', '~> 2.0.0']` means greater than or equal to 2 and grater than 2 but less than 3 so that is...
I dont think we do... @eheydrick am I missing something?
I am guessing that when `< 2.5` was added no one removed the `~> . 2.1`
I need to honestly play with this as I am not used to ever trying to support multiple major versions of a gem and usually support 1 major at a...
I will try to get to playing with this tonight.
@eheydrick that would make this break all the checks that rely on the aws-transport gem which I do think we need to do something about anyways as it's old and...
yes that was what I was saying here: https://github.com/sensu-plugins/sensu-plugins-elasticsearch/issues/48#issuecomment-300576773 My following comment when you said drop the it I somehow did not connect the dots that you only meant that...