Skip to content

Update otel dependencies - #318

Merged
Radhika Gupta (rads-1996) merged 3 commits into
microsoft:mainfrom
rads-1996:update-otel-deps-distro
Nov 5, 2026
Merged

Radhika Gupta (rads-1996) merged 3 commits into
microsoft:mainfrom
rads-1996:update-otel-deps-distro

Conversation

@rads-1996

Copy link
Copy Markdown
Member

No description provided.

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 dependency bumps leave uv.lock pinned below the new minimum versions with outdated dependency metadata, preventing uv sync --locked from accepting the committed lockfile.

1 open issue
What changed in this PR

Updates OpenTelemetry dependency minimums for the microsoft-opentelemetry distribution.

Changes:

  • Raises the API and SDK minimums from 1.45.0 to 1.45.2.
  • Raises ten instrumentation package minimums from 0.66b0 to 0.66b2, retaining the existing upper bounds.

Validation was limited to static inspection; dependency resolution and tests were not run.

File Description
pyproject.toml Updates API, SDK, and instrumentation dependency constraints.

Review effort: Balanced

Comment thread pyproject.toml Resolved
Copilot AI balanced review requested due to automatic review settings November 5, 2026 19:00
@github-actions

github-actions Bot commented Nov 5, 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 18,483.9 18,422.7 +0.33% ✅
azure_monitor_span yes 157,679.0 158,428.4 -0.47% ✅
otel_log no 21,214.3 21,377.6 -0.76% ✅
otel_span no 39,893.1 38,672.8 +3.16% ✅

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 dependency minimums were raised without regenerating uv.lock, causing locked syncs to fail and frozen installs to retain versions below the new minimums.

1 open issue

Review effort: Balanced

Copilot AI balanced review requested due to automatic review settings November 5, 2026 19:23

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.

🟢 Approval recommended

No active review issues remain for the OpenTelemetry dependency minimum updates, though dependency resolution and tests were not run.

No open issues 🎉

1 resolved since last review

Review effort: Balanced

@rads-1996
Radhika Gupta (rads-1996) merged commit eafdf06 into microsoft:main Nov 5, 2026
15 checks passed
@rads-1996
Radhika Gupta (rads-1996) deleted the update-otel-deps-distro branch November 5, 2026 21:02
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