Skip to content

Rework OTel exporter API and documentation#41705

Merged
alzimmermsft merged 53 commits into
mainfrom
jean/rework-otel-exporter-api-doc
Sep 20, 2024
Merged

Rework OTel exporter API and documentation#41705
alzimmermsft merged 53 commits into
mainfrom
jean/rework-otel-exporter-api-doc

Conversation

@jeanbisutti

Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

cc @lmolkova

@azure-sdk

azure-sdk commented Sep 3, 2024

Copy link
Copy Markdown
Collaborator

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
Comment thread sdk/monitor/azure-monitor-opentelemetry-exporter/README.md Outdated
Comment thread sdk/monitor/azure-monitor-opentelemetry-exporter/README.md Outdated
Comment thread sdk/monitor/azure-monitor-opentelemetry-exporter/README.md Outdated
@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
Copy Markdown
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
Copy Markdown
Member

/check-enforcer override

@alzimmermsft alzimmermsft merged commit d27c57c into main Sep 20, 2024
@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