Skip to content

Update telemetry SDK name to microsoft-opentelemetry#138

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

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

Conversation

@nikhilNava
Copy link
Copy Markdown
Collaborator

@nikhilNava nikhilNava commented May 12, 2026

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 telemetry.sdk.name attribute value used by the A365 telemetry implementation to align with this repository’s Python distribution name (microsoft-opentelemetry).

Changes:

  • Update TELEMETRY_SDK_NAME_VALUE from A365ObservabilitySDK to microsoft-opentelemetry in the A365 constants modules.
  • Keep both the a365/constants.py and a365/core/constants.py definitions in sync for the telemetry SDK name value.

Reviewed changes

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

File Description
src/microsoft/opentelemetry/a365/core/constants.py Updates the SDK name attribute value used by core A365 tracing to microsoft-opentelemetry.
src/microsoft/opentelemetry/a365/constants.py Updates the SDK name attribute value used by non-core A365 components to microsoft-opentelemetry.

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

Copy link
Copy Markdown
Member

@rads-1996 rads-1996 left a comment

Choose a reason for hiding this comment

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

LGTM

@nikhilNava nikhilNava enabled auto-merge (squash) May 12, 2026 22:15
@nikhilNava nikhilNava merged commit 542f1d9 into main May 12, 2026
12 checks passed
@nikhilNava nikhilNava deleted the nikhilc/update-sdk-name branch May 12, 2026 22:21
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.

5 participants