Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jul 28, 2025

This PR resolves the CI failure in Dependabot PR #92 by upgrading azure-monitor-opentelemetry-exporter from 1.0.0b26 to 1.0.0b29 along with compatible OpenTelemetry packages.

Problem

The original Dependabot PR failed CI because upgrading azure-monitor-opentelemetry-exporter to 1.0.0b29 created dependency compatibility issues with the existing opentelemetry-instrumentation-fastapi==0.46b0. This is a common pattern with OpenTelemetry packages where related components need to be updated together to maintain compatibility.

Solution

  • Upgraded azure-monitor-opentelemetry-exporter from 1.0.0b26 to 1.0.0b29 (the main goal)
  • Upgraded opentelemetry-instrumentation-fastapi from 0.46b0 to 0.48b0 for compatibility
  • Fixed ruff configuration deprecation warning by moving lint settings to [tool.ruff.lint] section

Benefits

The azure-monitor-opentelemetry-exporter 1.0.0b29 release includes:

  • Support for Python 3.12
  • Support for AAD Auth for live metrics
  • Allow passing custom TracerProvider for AzureMonitorTraceExporter
  • Various bug fixes and improvements

Verification

To verify the fix works:

python -m venv .venv
source .venv/bin/activate
python -m pip install --upgrade pip
python -m pip install -r requirements-dev.txt

All code syntax validation, linting (ruff), and formatting (black) checks pass with these changes.

Fixes #100.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits July 28, 2025 16:35
…ble dependencies

Co-authored-by: pamelafox <297042+pamelafox@users.noreply.github.com>
Co-authored-by: pamelafox <297042+pamelafox@users.noreply.github.com>
Copilot AI changed the title [WIP] Dependabot #92 to upgrade azure-monitor-opentelemetry-exporter failed CI Fix Dependabot azure-monitor-opentelemetry-exporter upgrade with compatible dependencies Jul 28, 2025
Copilot AI requested a review from pamelafox July 28, 2025 16:39
.gitignore Outdated
# Environments
.env
.venv
.venv2
Copy link
Owner

Choose a reason for hiding this comment

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

Remove this line, .venv2 is not a usual Python venv name

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed the .venv2 line from .gitignore. You're right that it's not a standard Python virtual environment name. (5d11562)

@pamelafox
Copy link
Owner

@copilot This PR no longer does an actual upgrade

@pamelafox pamelafox marked this pull request as ready for review July 29, 2025 05:17
Co-authored-by: pamelafox <297042+pamelafox@users.noreply.github.com>
Copilot AI requested a review from pamelafox July 29, 2025 05:20
@pamelafox pamelafox merged commit 0deaf46 into main Jul 29, 2025
3 checks passed
@pamelafox pamelafox deleted the copilot/fix-100 branch July 29, 2025 05:24
Copy link
Contributor Author

Copilot AI commented Jul 29, 2025

@copilot This PR no longer does an actual upgrade

This PR does perform the actual upgrade. The requirements.txt file shows:

Before: azure-monitor-opentelemetry-exporter==1.0.0b26
After: azure-monitor-opentelemetry-exporter==1.0.0b29

And also upgraded the compatible dependency:
Before: opentelemetry-instrumentation-fastapi==0.46b0
After: opentelemetry-instrumentation-fastapi==0.48b0

The upgrade was done in commit a9b02dd. Could you clarify what makes you think the upgrade isn't there?

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.

Dependabot #92 to upgrade azure-monitor-opentelemetry-exporter failed CI

2 participants