Skip to content

Conversation

trask
Copy link
Member

@trask trask commented Aug 29, 2025

@github-actions github-actions bot added the Monitor - Autoconfigure Monitor OpenTelemetry Autoconfigure label Aug 29, 2025
@trask trask force-pushed the semconv2 branch 3 times, most recently from f777681 to 0fea9fa Compare August 29, 2025 15:08
@trask trask marked this pull request as ready for review August 29, 2025 18:10
@Copilot Copilot AI review requested due to automatic review settings August 29, 2025 18:10
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates internal semantic convention classes in the Azure Monitor OpenTelemetry autoconfigure module to use v1.37.0 of the OpenTelemetry semantic conventions. The purpose is to maintain consistency with the latest OpenTelemetry standards while keeping the internal implementation isolated from external dependencies.

Key Changes

  • Replaces usage of a single monolithic SemanticAttributes class with specific semantic convention classes organized by domain (HTTP, DB, URL, etc.)
  • Adds new semantic convention classes for both stable and incubating attributes from OpenTelemetry v1.37.0
  • Updates imports and references throughout the codebase to use the new attribute classes

Reviewed Changes

Copilot reviewed 27 out of 28 changed files in this pull request and generated no comments.

Show a summary per file
File Description
AzureMonitorMetricExporterTest.java Updates import statements to use domain-specific semantic attribute classes
Multiple semconv files Adds new semantic convention attribute classes copied from OpenTelemetry v1.37.0
SpanDataMapper.java Updates all semantic attribute references to use new domain-specific classes
RequestChecker.java Updates messaging attribute reference
MetricDataMapper.java Updates HTTP and RPC attribute references
LogDataMapper.java Updates code and thread attribute references
AzureMonitorLogRecordExporter.java Updates exception attribute reference
checkstyle-suppressions.xml Adds suppressions for new semantic convention files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Monitor - Autoconfigure Monitor OpenTelemetry Autoconfigure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant