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

Check Tempo works with kiali with jaeger-query #5848

Closed
Tracked by #5850
aljesusg opened this issue Feb 16, 2023 · 16 comments
Closed
Tracked by #5850

Check Tempo works with kiali with jaeger-query #5848

aljesusg opened this issue Feb 16, 2023 · 16 comments
Assignees
Labels
backlog Triaged Issue added to backlog enhancement This is the preferred way to describe new end-to-end features. research Any task that is not related to a change in the codebase sub-task Ties an issue to an epic

Comments

@aljesusg
Copy link
Collaborator

Check Tempo works with kiali with jaeger-query

Related with #4278 (comment)

@aljesusg aljesusg added enhancement This is the preferred way to describe new end-to-end features. research Any task that is not related to a change in the codebase sub-task Ties an issue to an epic labels Feb 16, 2023
@iblancasa
Copy link

I can help with this

@aljesusg aljesusg mentioned this issue Feb 16, 2023
7 tasks
@aljesusg aljesusg added the backlog Triaged Issue added to backlog label Feb 16, 2023
@jshaughn
Copy link
Collaborator

@iblancasa Are you still planning to help here?

@batazor
Copy link

batazor commented Apr 17, 2023

@jshaughn At the moment, I don't see a problem with Kiali working with Grafana-tempo, but I think switching to open telemetry would be a good step in any case.

@iblancasa
Copy link

iblancasa commented Apr 17, 2023 via email

@iblancasa
Copy link

I created this repository to automatize my test: https://github.com/iblancasa/kiali-tempo-poc
There are some things that can be improved (like the Ingress for the Jaeger UI) but the important thing is tested.

What is needed to use Kiali with Grafana Tempo?

  1. I followed the instructions from the official Tempo website to deploy Grafana Tempo in a kind cluster.
  2. I had to do some modifications to:
    2.1. Decrease resource usage in order to be able to run the environment on my machine
    2.2. Expose the Zipkin collector
    2.3 Expose the GRPC Jaeger Query port
  3. When starting istio, specify the address where my Tempo Distributor component was deployed using --set meshConfig.defaultConfig.tracing.zipkin.address=distributor.tempo.svc.cluster.local:9411 (documentation)
  4. Specify the .external_services.tracing.in_cluster_url to the Jaeger Query GRPC endpoint .external_services.tracing.in_cluster_url to one Ingress route I created

Kiali showing traces from Tempo

I deployed the bookinfo application to ensure everything works properly.

Instead of exposing the Zipkin collector in Tempo, another option would be: to create an OpenTelemetry Collector to receive the traces and export them in OTLP format to Tempo.

@jshaughn if you want, I can try to simplify the automation from the repository and add it to the hack folder. Or just write some documentation somewhere if this comment is not enough.

Sorry for the delay and thanks!

@jshaughn
Copy link
Collaborator

Thanks @iblancasa! @aljesusg, how should we proceed here, documentation on the steps necessary to use Tempo, or something more formal?

@jshaughn
Copy link
Collaborator

Hi @iblancasa , I think what we will want to do is turn this information into content for kiali.io. Probably in this section: https://kiali.io/docs/configuration/p8s-jaeger-grafana/jaeger/.

@iblancasa
Copy link

Awesome. In that case, I'll create a PR to the kiali.io repository as soon as I can :)

@jshaughn
Copy link
Collaborator

Sounds great, if you need any guidance let us know.

@aljesusg aljesusg mentioned this issue Jul 4, 2023
@Hronom
Copy link

Hronom commented Jul 4, 2023

Please can we add it?

@Hronom
Copy link

Hronom commented Jul 6, 2023

Finally manage it to work, here is temporary workaround until support of Tempo not added directly to Kiali

@caerulescens
Copy link

I managed to get this all working with the latest version of the tempo-distributed chart by working from @Hronom and @iblancasa instructions. Thanks guys

@aljesusg
Copy link
Collaborator Author

Thabks to verify.
Anyway @josunect did a demo in kiali.io you will see it soon

You can check the preview here https://deploy-preview-686--kiali.netlify.app/docs/tutorials/tempo/

Please feel free to add more information about it 🙏

@caerulescens
Copy link

caerulescens commented Aug 20, 2023

Sure thing. I looked over the docs, and that's about the same thing I had to do. Create a PVC for MinIO and connect Tempo. Although, I'm doing it all through Helm managed by ArgoCD.

@aljesusg
Copy link
Collaborator Author

aljesusg commented Aug 20, 2023

Great !!! If you find any problem with argocd please let us know or if you prefer we can help you to contribute to the project with a tutorial about it 🤗

@aljesusg
Copy link
Collaborator Author

Added tutorial to kiali.io

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Triaged Issue added to backlog enhancement This is the preferred way to describe new end-to-end features. research Any task that is not related to a change in the codebase sub-task Ties an issue to an epic
Projects
Development

No branches or pull requests

6 participants