Skip to content

Migrate azure-monitor-opentelemetry-exporter to azure-json#41106

Merged
heyams merged 58 commits into
Azure:mainfrom
heyams:heya/azure-json
Aug 2, 2024
Merged

Migrate azure-monitor-opentelemetry-exporter to azure-json#41106
heyams merged 58 commits into
Azure:mainfrom
heyams:heya/azure-json

Conversation

@heyams

@heyams heyams commented Jul 11, 2024

Copy link
Copy Markdown
Member

No description provided.

@github-actions github-actions Bot added the OpenTelemetry OpenTelemetry instrumentation label Jul 11, 2024
@azure-sdk

azure-sdk commented Jul 11, 2024

Copy link
Copy Markdown
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

com.azure:azure-monitor-opentelemetry-exporter

@heyams heyams changed the title [azure-monitor-opentelemetry-exporter] migration from jackson.core.json to azure-json migration fazure-monitor-opentelemetry-exporter to azure-json Jul 12, 2024
@heyams heyams changed the title migration fazure-monitor-opentelemetry-exporter to azure-json Migrate azure-monitor-opentelemetry-exporter to azure-json Jul 12, 2024
@heyams heyams marked this pull request as ready for review July 20, 2024 00:10
@heyams heyams mentioned this pull request Jul 20, 2024
Merged
trask
trask reviewed Jul 30, 2024
View reviewed changes
Comment thread .../com/azure/monitor/opentelemetry/exporter/implementation/pipeline/TelemetryItemExporter.java
Comment thread ...-exporter/src/test/java/com/azure/monitor/opentelemetry/exporter/CustomValidationPolicy.java Outdated
Comment thread ...java/com/azure/monitor/opentelemetry/exporter/implementation/localstorage/Handle206Test.java Outdated
Comment thread ...java/com/azure/monitor/opentelemetry/exporter/implementation/localstorage/Handle206Test.java Outdated
Comment thread ...java/com/azure/monitor/opentelemetry/exporter/implementation/localstorage/Handle206Test.java Outdated
Comment thread ...java/com/azure/monitor/opentelemetry/exporter/implementation/localstorage/Handle206Test.java Outdated
Comment thread ...-exporter/src/test/java/com/azure/monitor/opentelemetry/exporter/CustomValidationPolicy.java Outdated
Comment thread ...porter/src/test/java/com/azure/monitor/opentelemetry/exporter/AzureMonitorStatsbeatTest.java Outdated
heyams
heyams commented Jul 31, 2024
View reviewed changes
Comment thread sdk/monitor/azure-monitor-opentelemetry-exporter/pom.xml
trask
trask reviewed Aug 1, 2024
View reviewed changes
Comment thread ...pentelemetry/exporter/implementation/localstorage/LocalStorageTelemetryPipelineListener.java Outdated
Comment thread ...pentelemetry/exporter/implementation/localstorage/LocalStorageTelemetryPipelineListener.java
Comment thread ...pentelemetry/exporter/implementation/localstorage/LocalStorageTelemetryPipelineListener.java Outdated
Comment thread ...pentelemetry/exporter/implementation/localstorage/LocalStorageTelemetryPipelineListener.java Outdated
Comment thread ...r/src/main/java/com/azure/monitor/opentelemetry/exporter/implementation/models/Response.java
Comment thread ...rc/test/java/com/azure/monitor/opentelemetry/exporter/AzureMonitorExportersEndToEndTest.java Outdated
Comment thread ...rc/test/java/com/azure/monitor/opentelemetry/exporter/AzureMonitorExportersEndToEndTest.java Outdated
Comment thread ...porter/src/test/java/com/azure/monitor/opentelemetry/exporter/AzureMonitorStatsbeatTest.java Outdated
Comment thread ...porter/src/test/java/com/azure/monitor/opentelemetry/exporter/AzureMonitorStatsbeatTest.java Outdated
Comment thread ...porter/src/test/java/com/azure/monitor/opentelemetry/exporter/AzureMonitorStatsbeatTest.java Outdated
@heyams heyams merged commit ce56bbf into Azure:main Aug 2, 2024
@trask trask mentioned this pull request Oct 18, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alzimmermsft alzimmermsft alzimmermsft left review comments

@trask trask trask approved these changes

@ramthi ramthi Awaiting requested review from ramthi ramthi is a code owner

@harsimar harsimar Awaiting requested review from harsimar harsimar is a code owner

+1 more reviewer

@jeanbisutti jeanbisutti jeanbisutti left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

OpenTelemetry OpenTelemetry instrumentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@heyams @azure-sdk @trask @jeanbisutti @alzimmermsft