Skip to content

chore: disable Azure SDK telemetry in Application Insights configuration #331

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Harmanpreet-Microsoft wants to merge 6 commits into
base: dev
Choose a base branch
from
Open

chore: disable Azure SDK telemetry in Application Insights configuration #331

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
d714033
fix: disable Azure SDK auto-instrumentation to prevent response forma…
Jan 22, 2026
87653f0
Update logging message for Application Insights configuration
Harmanpreet-Microsoft Jan 27, 2026
2f5ca13
fix: enable public network access and local authentication for WAF co…
Harmanpreet-Microsoft Feb 25, 2026
99d2742
Merge branch 'dev' into psl-wafissue
Harmanpreet-Microsoft Feb 25, 2026
0088b6a
fix: uncomment Azure Monitor OpenTelemetry integration in API routes
Harmanpreet-Microsoft Feb 26, 2026
540c1f3
fix: correct formatting in Azure Monitor configuration options
Harmanpreet-Microsoft Feb 26, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions infra/main.bicep
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -234,8 +234,8 @@ module logAnalyticsWorkspace 'br/public:avm/res/operational-insights/workspace:0
}
: null
// WAF aligned configuration for Private Networking
publicNetworkAccessForIngestion: enablePrivateNetworking ? 'Disabled' : 'Enabled'
publicNetworkAccessForQuery: enablePrivateNetworking ? 'Disabled' : 'Enabled'
publicNetworkAccessForIngestion: 'Enabled'
publicNetworkAccessForQuery: 'Enabled'
dataSources: enablePrivateNetworking
? [
{
Expand Down Expand Up @@ -291,7 +291,7 @@ module applicationInsights 'br/public:avm/res/insights/component:0.7.0' = if (en
retentionInDays: 365
kind: 'web'
disableIpMasking: false
disableLocalAuth: true
disableLocalAuth: false
flowType: 'Bluefield'
}
}
Expand Down
10 changes: 7 additions & 3 deletions src/backend/api/api_routes.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@

# Third-party
# Azure Monitor OpenTelemetry integration is currently causing issues with OpenAI calls in process_batch_async, needs further investigation, commenting out for now
# from azure.monitor.opentelemetry import configure_azure_monitor
from azure.monitor.opentelemetry import configure_azure_monitor

from common.logger.app_logger import AppLogger
from common.services.batch_service import BatchService
Expand Down Expand Up @@ -44,8 +44,12 @@
instrumentation_key = os.getenv("APPLICATIONINSIGHTS_CONNECTION_STRING")
if instrumentation_key:
# Configure Application Insights if the Instrumentation Key is found
# commenting below line as configure_azure_monitor is causing issues with OpenAI calls in process_batch_async, needs further investigation
# configure_azure_monitor(connection_string=instrumentation_key)
configure_azure_monitor(
connection_string=instrumentation_key,
instrumentation_options={
"azure_sdk": {"enabled": False},
}
)
logging.info(
"Application Insights configured with the provided Instrumentation Key"
)
Expand Down