Skip to content

Comments

Revert packages to fix no logs below Warning#5756

Merged
premun merged 1 commit intodotnet:mainfrom
premun:prvysoky/logs
Jan 12, 2026
Merged

Revert packages to fix no logs below Warning#5756
premun merged 1 commit intodotnet:mainfrom
premun:prvysoky/logs

Conversation

@premun
Copy link
Member

@premun premun commented Jan 12, 2026

Introduced in #5707

The service shows no logs below Warning level in AppInsights

Copilot AI review requested due to automatic review settings January 12, 2026 15:56
Copy link
Contributor

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 reverts Azure Monitor OpenTelemetry packages from stable to beta versions to fix a logging issue introduced in #5707, where logs below Warning level were not appearing. The PR also updates several OpenTelemetry packages to version 1.14.0 for consistency.

Changes:

  • Reverts Azure.Monitor.OpenTelemetry.AspNetCore and Azure.Monitor.OpenTelemetry.Exporter to beta versions (1.4.0-beta.1 and 1.5.0-beta.1 respectively)
  • Updates OpenTelemetry packages to 1.14.0 (OpenTelemetry.Exporter.OpenTelemetryProtocol, OpenTelemetry.Instrumentation.GrpcNetClient, OpenTelemetry.Instrumentation.Runtime)
  • Updates Aspire.AppHost.Sdk from 9.2.0 to 13.1.0

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/ProductConstructionService/ProductConstructionService.AppHost/ProductConstructionService.AppHost.csproj Updates Aspire.AppHost.Sdk version from 9.2.0 to 13.1.0
Directory.Packages.props Reverts Azure Monitor OpenTelemetry packages to beta versions and upgrades OpenTelemetry packages to 1.14.0

@premun premun enabled auto-merge (squash) January 12, 2026 16:03
@premun premun merged commit df7ec0d into dotnet:main Jan 12, 2026
13 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants