Skip to content

Update Otel dependencies - #283

Merged
Radhika Gupta (rads-1996) merged 3 commits into
microsoft:mainfrom
rads-1996:update-to-latest-package-versions
Oct 7, 2026
Merged

Radhika Gupta (rads-1996) merged 3 commits into
microsoft:mainfrom
rads-1996:update-to-latest-package-versions

Conversation

@rads-1996

Copy link
Copy Markdown
Member

No description provided.

@github-actions

github-actions Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Performance comparison

Threshold: regressions >15.0% on gating scenarios fail the build. Higher ops/s is better; positive Δ means the PR is slower.

Scenario Gating Baseline (ops/s) Candidate (ops/s) Δ % Status
azure_monitor_log yes 19,582.9 19,606.3 -0.12% ✅
azure_monitor_span yes 160,978.8 158,428.4 +1.61% ✅
otel_log no 22,772.8 22,669.6 +0.46% ✅
otel_span no 41,504.1 40,741.5 +1.87% ✅

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The committed uv.lock remains incompatible with the updated dependency requirements.

1 open finding
What changed in this PR

Updates the distribution’s OpenTelemetry and Azure dependency requirements.

Changes:

  • Moves OpenTelemetry packages to 1.45 and instrumentation to 0.66b0.
  • Raises minimum versions for Azure Core, the Azure Monitor exporter, and the Azure resource detector.
File Description
pyproject.toml Updates OpenTelemetry and Azure dependency constraints.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread pyproject.toml
@rads-1996 Radhika Gupta (rads-1996) added the skip-changelog Not required for changes to test files, github actions, etc. label Oct 7, 2026
Copilot AI balanced review requested due to automatic review settings October 7, 2026 19:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The committed lockfile still specifies OpenTelemetry SDK 1.44 and needs regeneration to match the updated dependencies.

0 open findings

1 resolved since last review

🧠 Review effort: Balanced

@rads-1996
Radhika Gupta (rads-1996) merged commit 502a644 into microsoft:main Oct 7, 2026
12 checks passed
@rads-1996
Radhika Gupta (rads-1996) deleted the update-to-latest-package-versions branch October 7, 2026 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Not required for changes to test files, github actions, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants