Skip to content

fix: Reverting the open telemetry changes in api_routes.py#124

Merged
Roopan-Microsoft merged 1 commit intomainfrom
psl-revert-opentelemetry
Jun 6, 2025
Merged

fix: Reverting the open telemetry changes in api_routes.py#124
Roopan-Microsoft merged 1 commit intomainfrom
psl-revert-opentelemetry

Conversation

@Abdul-Microsoft
Copy link
Copy Markdown
Contributor

Purpose

This pull request removes telemetry and logging-related functionality from the src/backend/api/api_routes.py file, simplifying the codebase and reducing dependencies. The most significant changes include removing Application Insights configuration, eliminating the use of OpenTelemetry for tracing, and removing event tracking calls throughout the API endpoints.

Removal of telemetry and logging functionality:

  • Removed Application Insights configuration and logging setup, including suppression of specific loggers and the use of environment variables for instrumentation keys. (src/backend/api/api_routes.py)
  • Eliminated all OpenTelemetry tracing code, including span creation, exception recording, and status setting in API endpoints. (src/backend/api/api_routes.py) [1] [2] [3]

Removal of event tracking:

  • Removed calls to track_event_if_configured across multiple endpoints, such as start_processing, download_files, batch_status_updates, and others. These calls previously logged events like "ProcessingStart" and "FileUploaded". (src/backend/api/api_routes.py) [1] [2] [3] [4]

General cleanup:

  • Removed unused imports related to telemetry and logging, such as opentelemetry, azure.monitor.opentelemetry, and logging. (src/backend/api/api_routes.py)
  • Removed redundant blank lines and minor formatting adjustments to improve code readability. (src/backend/api/api_routes.py) [1] [2] [3]

These changes streamline the code by removing dependencies and functionality that are no longer required, making the API routes easier to maintain.

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • ...

Other Information

@Roopan-Microsoft Roopan-Microsoft merged commit d271088 into main Jun 6, 2025
5 of 6 checks passed
@Roopan-Microsoft Roopan-Microsoft deleted the psl-revert-opentelemetry branch June 6, 2025 15:09
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.4.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Ritesh-Microsoft pushed a commit that referenced this pull request Oct 10, 2025
fix: Reverting the open telemetry changes in api_routes.py
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.

2 participants