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

feat: Otel TracingHook implementation and repo skeleton #1

Merged
merged 2 commits into from
Apr 22, 2024

Conversation

tanderson-ld
Copy link
Contributor

@tanderson-ld tanderson-ld commented Apr 10, 2024

Requirements

  • I have added test coverage for new or changed functionality
  • I have followed the repository's pull request submission guidelines
  • I have validated my changes against all supported platform versions

Related issues

236794

Describe the solution you've provided

Implementation of SDK Hook interface that records spans and events for feature flag evaluations using the OpenTelemetry API.

@tanderson-ld
Copy link
Contributor Author

Still needs a top level README.

@tanderson-ld tanderson-ld requested a review from a team April 10, 2024 20:00
@tanderson-ld tanderson-ld requested a review from kinyoklion April 18, 2024 15:18
@@ -0,0 +1,2 @@
# Repository Maintainers
* @launchdarkly/team-sdk-java
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This says it is missing permissions at the moment, so you may need to double check the repo labels versus the team requirements.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the query is query = "org:launchdarkly topic:launchdarkly-sdk language:Java" , and I think the language tag is automatically derived. Does that sound plausible? If so, I think we have to merge something before this will know the repo is for java.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It does sound plausible. You may need to trigger a terraform run after merging this.

@tanderson-ld tanderson-ld merged commit c808f7c into main Apr 22, 2024
1 check passed
@tanderson-ld tanderson-ld deleted the ta/sc-236794/otel-package-squashed branch April 22, 2024 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants