Skip to content

Conversation

@Abdul-Microsoft
Copy link
Contributor

Purpose

  • This pull request re-enables Azure Application Insights monitoring in the app_kernel.py file by uncommenting previously disabled code. The changes ensure that telemetry data is sent to Azure when the appropriate connection string is provided.

Azure Application Insights monitoring:

  • src/backend/app_kernel.py: Re-enabled the import of configure_azure_monitor from azure.monitor.opentelemetry by uncommenting the import statement.
  • src/backend/app_kernel.py: Re-enabled the call to configure_azure_monitor with the connection_string parameter, ensuring that Application Insights is configured when a connection string is available.

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@Roopan-Microsoft Roopan-Microsoft merged commit 50945d1 into dev Jul 1, 2025
7 checks passed
@Abdul-Microsoft Abdul-Microsoft deleted the psl-fix-telemetryissue branch July 16, 2025 04:57
@github-actions
Copy link

🎉 This PR is included in version 2.1.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

blessing-msft pushed a commit that referenced this pull request Aug 1, 2025
fix: enable the appinsights telemetry logs code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants