opentracing-java icon indicating copy to clipboard operation
opentracing-java copied to clipboard

Published 20 hours ago • verified publisher icon opentracing

Android support

Open jpkrohling opened this issue 6 years ago • 9 comments

We currently don't have reports about applications using OpenTracing with Android. It would then be worth doing a research on what's needed to get it working. At this moment, we are not sure what would break and what's missing.

Task outline:

  • [ ] Bootstrap an Android project
  • [ ] Instrument the code manually using the OpenTracing libraries
  • [ ] Assert that the instrumentation works by using the MockTracer
  • [ ] Report the findings

jpkrohling avatar Aug 17 '18 16:08 jpkrohling

@jpkrohling We have to make a sample android app and instrument the code using open Tracing libraries?

SIMRAN1 avatar Sep 21 '18 15:09 SIMRAN1

Yes, go as far as you can, with the main goal being to be able to visualize a trace in Jaeger.

The application itself isn't that important, it can be just a simple app with a single button in the UI. Clicking the button would generate a span using OpenTracing APIs. Then, plug in a concrete tracer like Jaeger's and see how this span could reach the collector and be available in Jaeger's UI.

I don't expect it all to work, but it would be interesting to see how far we can go without changes to the existing libraries.

jpkrohling avatar Sep 21 '18 15:09 jpkrohling

Thanks @jpkrohling! I will work on this.

SIMRAN1 avatar Sep 22 '18 03:09 SIMRAN1

Hie @jpkrohling, Can I also work on this issue?

SaumiaSinghal avatar Sep 23 '18 19:09 SaumiaSinghal

Hie @jpkrohling, I have a doubt. We should not be using LightStep Tracers for this purpose, we should only use Jaeger Tracer. Right?

SaumiaSinghal avatar Sep 23 '18 23:09 SaumiaSinghal

@SaumiaSinghal you can absolutely work on this one as well!

I wasn't aware of LightStep's support for Android before, but I'd say that it's certainly OK to follow their instructions on a first step, as it seems it's supposed to work already (unless @tedsuo thinks it's not appropriate). Then, replace the LS tracer with MockTracer and try it out. If you can get all of that to work, then the next possible step is to replace the MockTracer with Jaeger's.

If you follow this path, I'm sure @tedsuo would appreciate if you'd include LS in your final report.

jpkrohling avatar Sep 24 '18 07:09 jpkrohling

@SIMRAN1, @SaumiaSinghal , if you have progress to report on this issue, please do so before the deadline, which is tomorrow Tuesday, October 30, 2018 4pm UTC.

https://www.outreachy.org/december-2018-march-2019-outreachy-internships/communities/cncf-tracing/research-the-gaps-in-tracing-web-clients/contributions/

jpkrohling avatar Oct 29 '18 17:10 jpkrohling

Any update on this?

Bikraji avatar Nov 01 '20 16:11 Bikraji

Unfortunately, no. Also, I don't think we'll make progress on this, given that work is now centered around OpenTelemetry.

jpkrohling avatar Nov 02 '20 08:11 jpkrohling