Skip to content

Bump azure-core-version to match the exporter changes - #274

Closed
Radhika Gupta (rads-1996) wants to merge 2 commits into
microsoft:mainfrom
rads-1996:bump-azure-core
Closed

Radhika Gupta (rads-1996) wants to merge 2 commits into
microsoft:mainfrom
rads-1996:bump-azure-core

Conversation

@rads-1996

@rads-1996 Radhika Gupta (rads-1996) commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Will merge once the exporter version with the same bump is released to avoid conflicts.

@github-actions

github-actions Bot commented Sep 25, 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 41,380.5 41,498.9 -0.29% ✅
azure_monitor_span yes 311,090.4 309,119.0 +0.64% ✅
otel_log no 49,625.3 49,825.6 -0.40% ✅
otel_span no 90,522.3 89,710.2 +0.91% ✅

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.

Copilot review overview

🟡 Changes recommended

Regenerate uv.lock to keep dependency metadata consistent.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 Medium severity

Open (1)
What changed in this PR

Raises the minimum azure-core version to align with exporter changes.

Changes:

  • Updates the lower bound from 1.28.0 to 1.38.0.
  • Requires regenerating uv.lock to reflect the updated constraint.
File Summary
pyproject.toml Updates the azure-core dependency constraint. A moderate issue remains: regenerate uv.lock.

💡 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 Sep 25, 2026
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.

2 participants