Skip to content
This repository has been archived by the owner on Dec 1, 2018. It is now read-only.

Any plans to support Elasticsearch 5 #1410

Closed
maxott opened this issue Dec 12, 2016 · 6 comments
Closed

Any plans to support Elasticsearch 5 #1410

maxott opened this issue Dec 12, 2016 · 6 comments

Comments

@maxott
Copy link

maxott commented Dec 12, 2016

It appears that 1.2 can't connect to ES5

E1212 05:24:11.987800 120 factory.go:75] Failed to create sink: Failed to create ElasticSearch client: no Elasticsearch node available

It works perfectly fine for ES2. Unfortunately, some of my other components require ES5.

Any current plans to upgrade the ES backend?

@AlmogBaku
Copy link
Contributor

Upgrading to ES5 will break the current support of ES2...
I think that for the time being we need to support both of the sinks version..

Also, It looks like we'll have to replace only the ES driver and not the eventer/heapster sinks, since it's modular.

Currently I don't have the time/resources to build the ansible script to install ES5, so I'll have to wait for ES team to update the script in order to create a new ES Cluster..

@rikatz
Copy link
Contributor

rikatz commented Dec 26, 2016

@AlmogBaku Why it will break support to ES2? I think it's only something like pointing the lib contained here: https://github.com/kubernetes/heapster/blob/master/common/elasticsearch/elasticsearch.go#L25

To the v5 library, right?

I'll try to change the source code pointing to this new version, and see what happens. We did this earlier with telegraf and worked fine, the only real problem was with fields containing '.' character, because ES 2 does not support them.

@rikatz
Copy link
Contributor

rikatz commented Dec 26, 2016

Guys, it's pretty simple to support Elastic v5. I did some simple tests here and it worked. I'm going to open a PR late tomorrow, to change the lib version (and some minor changes needed to make this work) and post here.

@AlmogBaku
Copy link
Contributor

olivere/elastic#431

@AlmogBaku
Copy link
Contributor

AlmogBaku commented Jan 5, 2017

@rikatz the discussion on the issue above means that we need to create a different sink for es5; however all the drivers can be the same with the current implementation since they abstract

@rikatz
Copy link
Contributor

rikatz commented Aug 16, 2017

@maxott Elasticsearch 5 is already supported in Heapster (as merged in #1607).

Can you please close this issue?

Thanks!

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

No branches or pull requests

4 participants