Skip to content

Fix issues with enabling application insights logging - #929

Merged
Jesper Fajers (Jefajers) merged 1 commit into
Azure:mainfrom
StefanIvemo:appinsight-fix
Aug 25, 2025
Merged

Jesper Fajers (Jefajers) merged 1 commit into
Azure:mainfrom
StefanIvemo:appinsight-fix

Conversation

@StefanIvemo

@StefanIvemo Stefan Ivemo (StefanIvemo) commented Aug 22, 2025

Copy link
Copy Markdown
Collaborator

Overview/Summary

Fixed a typo for the secret name APPLICATIONINSIGHTS_CONNECTIONSTRING .

This PR fixes/adds/changes/removes

This PR fixes #928

Breaking Changes

N/A

Testing Evidence

N/A

As part of this Pull Request I have

  • Checked for duplicate Pull Requests
  • Associated it with relevant issues, for tracking and closure.
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Performed testing and provided evidence.
  • Updated relevant and associated documentation.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes a typo in an environment variable name used for Application Insights logging configuration. The variable name was incorrectly using underscores as APPLICATIONINSIGHTS_CONNECTION_STRING instead of the correct APPLICATIONINSIGHTS_CONNECTIONSTRING.

  • Fixed environment variable name typo for Application Insights connection string

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Jefajers Jesper Fajers (Jefajers) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ah good catch Stefan Ivemo (@StefanIvemo) 🥇.

@Jefajers Jesper Fajers (Jefajers) moved this from In progress to Pending release in AzOps Aug 25, 2025
@Jefajers
Jesper Fajers (Jefajers) merged commit 5560186 into Azure:main Aug 25, 2025
4 checks passed
@Jefajers Jesper Fajers (Jefajers) moved this from Pending release to Done in AzOps Sep 16, 2025

This branch was previously deployed

1 inactive deployment
test d6b751cd Deployed Aug 25, 2025 by StefanIvemo via Tests - Subscription Only #358
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Pipelines are failing after enabling Application Insights logging

3 participants