Skip to content
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.

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

Integrate Application Insights connection string with Azure Container App #245

Conversation

tjementum
Copy link
Member

Summary & Motivation

Add the Application Insights connection string as an environment variable, named APPLICATIONINSIGHTS_CONNECTION_STRING, to the Azure Container App. This is a special name that enables the Application Insights SDK to automatically recognize and use this connection string, eliminating the need for additional configuration.

Opting not to retrieve the connection string via Bicep due to a known issue (see GitHub Issue #157), which affects the --what-if functionality. Instead, the Azure CLI is used to fetch the connection string and pass it as a parameter to Bicep, ensuring the --what-if functionality remains effective.

To enhance clarity, the environmentName and environmentId parameters in the Bicep container App module have been renamed to containerAppsEnvironmentName and containerAppsEnvironmentId, respectively.

Checklist

  • I have added a Label to the pull-request
  • I have added tests, and done manual regression tests
  • I have updated the documentation, if necessary

…me to containerAppsEnvironmentName for clarity
…icep param to configure Container App Environment variable
@tjementum tjementum added the Enhancement New feature or request label Dec 2, 2023
@tjementum tjementum merged commit acc0f01 into main Dec 2, 2023
4 checks passed
@tjementum tjementum deleted the add-application-insights-connection-string-environment-variable branch December 2, 2023 18:17
@tjementum tjementum self-assigned this Feb 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ensure Application Insights connection string is automatically configured for container apps
1 participant