open-telemetry / opentelemetry-android

OpenTelemetry Tooling for Android
Apache License 2.0
147 stars 33 forks source link

Instrumentation API part 2 #408

Closed LikeTheSalad closed 4 months ago

LikeTheSalad commented 4 months ago

These changes involve the telemetry sent around network status changes. The amount of files is large mostly because of several files, prod code and unit tests, moved over to android-agent (to become part of a service).

Since the relationship around the instrumentations and the android-agent module is going to be inverted and a couple of tools needed across multiple modules will be turned into services, I realized that it's going to be very difficult to keep the build passing without having all the instrumentation modules already migrated over to the new API. Also, as soon as the first instrumentation is migrated (like this one), the android-agent will no longer provide that functionality out of the box, as now the dependency relationship is inverted, which breaks the current behavior, so in order to avoid causing troubles while all the instrumentation modules are migrated, I've created a feature branch to merge all the PRs into, one instrumentation at a time, so that they are easier to review and won't break the current functionality.