Skip to content

Update telemetry SDK name to microsoft-opentelemetry#134

Merged
nikhilNava merged 1 commit into
mainfrom
nikhilc/update-sdk-name
May 12, 2026
Merged

Update telemetry SDK name to microsoft-opentelemetry#134
nikhilNava merged 1 commit into
mainfrom
nikhilc/update-sdk-name

Conversation

@nikhilNava
Copy link
Copy Markdown
Collaborator

Change TELEMETRY_SDK_NAME_VALUE from A365ObservabilitySDK to microsoft-opentelemetry to align with the package distribution name.

Sibling PRs:

Change TELEMETRY_SDK_NAME_VALUE from 'A365ObservabilitySDK' to
'microsoft-opentelemetry' to align with the package distribution name.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

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 the A365-specific telemetry.sdk.name value used when enriching A365 spans, changing it from a baseline-specific identifier (A365ObservabilitySDK) to the distro/product identifier (microsoft-opentelemetry) to align with the package distribution naming.

Changes:

  • Update OpenTelemetryConstants.TELEMETRY_SDK_NAME_VALUE to microsoft-opentelemetry for A365 span enrichment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/a365/constants.ts
Comment on lines +137 to 139
public static readonly TELEMETRY_SDK_NAME_VALUE = "microsoft-opentelemetry";
public static readonly TELEMETRY_SDK_LANGUAGE_VALUE = "nodejs";
public static readonly TELEMETRY_SDK_VERSION_VALUE = MICROSOFT_OPENTELEMETRY_VERSION;
@nikhilNava nikhilNava merged commit c8187b0 into main May 12, 2026
9 checks passed
@nikhilNava nikhilNava deleted the nikhilc/update-sdk-name branch May 12, 2026 22:13
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.

4 participants