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

Does Jaeger OpenTelemetry support Elasticsearch 7? #2407

Closed
hongbo-miao opened this issue Aug 22, 2020 · 3 comments · Fixed by #2409
Closed

Does Jaeger OpenTelemetry support Elasticsearch 7? #2407

hongbo-miao opened this issue Aug 22, 2020 · 3 comments · Fixed by #2409
Labels

Comments

@hongbo-miao
Copy link

hongbo-miao commented Aug 22, 2020

In current Jaeger OpenTelemetry docker compose example
https://github.com/jaegertracing/jaeger/blob/master/crossdock/jaeger-opentelemetry-docker-compose.yml

it is using Elasticsearch 6:

image: docker.elastic.co/elasticsearch/elasticsearch-oss:6.8.3

I know Jaeger supports Elasticsearch 7.

However, this demo since it is using OpenTelemetry version images for Jaeger:

  • jaegertracing/jaeger-opentelemetry-collector
  • jaegertracing/jaeger-opentelemetry-agent

I succeed testing on elasticsearch-oss 6.8.3 and 6.8.12, but failed upgrading to 7.0.0 and 7.9.0, got error:

{
  "data": null,
  "total": 0,
  "limit": 0,
  "offset": 0,
  "errors": [
    {
      "code": 500,
      "msg": "search services failed: elastic: Error 400 (Bad Request): all shards failed [type=search_phase_execution_exception]"
    }
  ]
}

I tried to upgrading to Elasticsearch 7 for a while, but no luck.
So just wonder does Jaeger OpenTelemetry currently support Elasticsearch 7? cc author @pavolloffay

If it does, it would be very helpful to upgrade the example docker-compose.yml to use Elasticsearch 7. Thanks!

@pavolloffay
Copy link
Member

hi @hongbo-miao the Jaeger OTEL and main Jaeger version support ES 5.x, 6.x and 7.x.

There was a bug, I have submitted a fix #2409. Tve version bump in compose and integration tests can be done in a separate PR.

@hongbo-miao
Copy link
Author

Thank you sooo much for the help @pavolloffay !
Really appreciate your time and the contributions on Jaeger OTEL

@hongbo-miao
Copy link
Author

Confirmed, test latest images, and works with elasticsearch-oss:7.9.0
Thanks a lot!

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

Successfully merging a pull request may close this issue.

2 participants