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

Integrate with opentelemetry-android #1186

Conversation

surbhiia
Copy link
Contributor

@surbhiia surbhiia commented Mar 7, 2025

This PR covers the following:

  • Setup for installing opentelemetry-android instrumentations
  • Integrated HttpURLConnection auto-instrumentation
  • Upgrades to necessary libraries as needed for the integration:
    • Desugar lib
    • Bytebuddy version
    • oTel semantic conventions
  • Forcing earlier versions of remaining to maintain compatibility with our SDK
    • kotlin-stdlib (1.8.0 - our SDK requirement)
    • androidx.core:core (last most used stable version 1.31.1)
    • androidx.core:core-ktx (last most used stable version 1.31.1)
  • Updates needed in plugin code and exsuring it’s usage in sample app and testing telemetry generation
  • Upgrades to certain API names to tell them apart from similar APIs coming from upstream like:
    • OpenTelemtry -> SplunkRumOpenTelemtrySdk
    • SessionManager -> SplunkSessionManager (this will be replaced later anyway when upstream SessionManager is used instead)

Upcoming in future PRs:

  • Based on one/two plugin approach we decide - integrate okHttp3 auto-instrumentation along with the changes needed for that.
  • Add additional attributes once atrribute extractors PR upstream is merged.
  • Validate all attributes for Network Request spans once we have all of the SDK setup for attributes in place.

@surbhiia surbhiia requested a review from a team as a code owner March 7, 2025 20:08
@aditi-s3 aditi-s3 closed this Mar 18, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Mar 18, 2025
@aditi-s3 aditi-s3 reopened this Mar 18, 2025
@aditi-s3 aditi-s3 closed this Mar 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants