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

Rework OTel exporter API and documentation #41705

Merged
merged 53 commits into from
Sep 20, 2024

Conversation

jeanbisutti
Copy link
Contributor

No description provided.

@jeanbisutti jeanbisutti marked this pull request as draft September 3, 2024 13:57
@github-actions github-actions bot added the OpenTelemetry OpenTelemetry instrumentation label Sep 3, 2024
@jeanbisutti
Copy link
Contributor Author

cc @lmolkova

@azure-sdk
Copy link
Collaborator

azure-sdk commented Sep 3, 2024

API change check

API changes are not detected in this pull request.

@jeanbisutti jeanbisutti force-pushed the jean/rework-otel-exporter-api-doc branch from d4d3bb7 to 4fc4fc9 Compare September 4, 2024 12:10
@jeanbisutti jeanbisutti force-pushed the jean/rework-otel-exporter-api-doc branch from 810782d to c3bb22d Compare September 4, 2024 15:59
@jeanbisutti
Copy link
Contributor Author

@lmolkova

AzureMonitor provides a high-level API to enable Azure Monitor for OpenTelemetry autoconfiguration:

AutoConfiguredOpenTelemetrySdkBuilder sdkBuilder = AutoConfiguredOpenTelemetrySdk.builder();

OpenTelemetry openTelemetry = new AzureMonitor().connectionString("{connection-string}")
                                      .configure(sdkBuilder).build().getOpenTelemetrySdk();

We expect that most users will use the OpenTelemetry autoconfiguration.

AzureMonitorExporterBuilder provides a low-level API to create OpenTelemetry span, log record and metric exporters for Azure.

There is a CI compilation issue with TelemetryHelper. It seems that the CI compiles this class for the OpenTelemetry exporter 1.0.0-beta.28, but also with the exporter version of the main branch. The exporter API of the main branch has changed, leading to a compilation failure. Any help to be able to fix this CI issue would be appreciated.

@jeanbisutti jeanbisutti force-pushed the jean/rework-otel-exporter-api-doc branch 2 times, most recently from 2aaf035 to 43d215b Compare September 11, 2024 12:05
@jeanbisutti jeanbisutti force-pushed the jean/rework-otel-exporter-api-doc branch from a590ee5 to db7b4f0 Compare September 16, 2024 09:54
jeanbisutti and others added 23 commits September 20, 2024 15:09
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
…yle-suppressions.xml

Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
@jeanbisutti jeanbisutti force-pushed the jean/rework-otel-exporter-api-doc branch from 35ee2d2 to f335fa0 Compare September 20, 2024 13:23
@alzimmermsft
Copy link
Member

/check-enforcer override

@alzimmermsft alzimmermsft merged commit d27c57c into main Sep 20, 2024
95 of 103 checks passed
@alzimmermsft alzimmermsft deleted the jean/rework-otel-exporter-api-doc branch September 20, 2024 14:47
mssfang pushed a commit to mssfang/azure-sdk-for-java that referenced this pull request Oct 21, 2024
Rework OTel exporter API and documentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OpenTelemetry OpenTelemetry instrumentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants