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

[Fun task]: Add React tracing #1307

Open
yurishkuro opened this issue Mar 27, 2023 · 4 comments · May be fixed by #1627
Open

[Fun task]: Add React tracing #1307

yurishkuro opened this issue Mar 27, 2023 · 4 comments · May be fixed by #1627

Comments

@yurishkuro
Copy link
Member

Requirement

Understand and visualize the behavior of Jaeger UI via traces

Problem

Currently we only start traces at the query service.

Proposal

Perhaps follow suggestions in the blog?

https://developers.redhat.com/articles/2023/03/22/how-enable-opentelemetry-traces-react-applications

Open questions

How can we send trace data from the browser to Jaeger backend? Should the query-service implement a dedicated endpoint where frontend can report traces?

@psk001
Copy link

psk001 commented Jul 28, 2023

Hi @yurishkuro , is it up for grab can I start on it ?

@yurishkuro
Copy link
Member Author

@psk001 yes

@psk001
Copy link

psk001 commented Jul 31, 2023

Thanks @yurishkuro. I followed the mentioned article and was able to get traces using jaeger all-in-one For the exporter, a URL is required to send traces to collector

const collectorExporter = new OTLPTraceExporter({
// url:""
  headers: {}
});

Without a given value, it defaults to localhost. Is there a collector URL that can be added here?

@yurishkuro
Copy link
Member Author

All-in-one listens on port 4318 for OTLP over HHTP.

@psk001 psk001 linked a pull request Aug 1, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants