Skip to content

Conversation

@StefanIvemo
Copy link
Collaborator

@StefanIvemo StefanIvemo commented Aug 22, 2025

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.

Copy link

Copilot AI left a comment

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 Jefajers self-assigned this Aug 25, 2025
@Jefajers Jefajers added the bug Something isn't working label Aug 25, 2025
@Jefajers Jefajers added this to the v2.8.2 milestone Aug 25, 2025
@Jefajers Jefajers moved this to In progress in AzOps Aug 25, 2025
Copy link
Member

@Jefajers Jefajers left a comment

Choose a reason for hiding this comment

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

Ah good catch @StefanIvemo 🥇.

@Jefajers Jefajers moved this from In progress to Pending release in AzOps Aug 25, 2025
@Jefajers Jefajers merged commit 5560186 into Azure:main Aug 25, 2025
4 checks passed
@Jefajers 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

2 participants