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

Fix issues with enabling application insights logging#929
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
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