Skip to content

Fix AOT warning regression in Azure.Monitor.OpenTelemetry.Exporter #56368

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.

Sign up for GitHub

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

Merged
rajkumar-rangaraj merged 2 commits into from
Feb 20, 2026
+8 −9
Merged

Fix AOT warning regression in Azure.Monitor.OpenTelemetry.Exporter #56368

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
bc1e1d4
Fix AOT warning regression in Azure.Monitor.OpenTelemetry.Exporter
eerhardt Feb 20, 2026
77546e3
Respond to PR feedback
eerhardt Feb 20, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion ...itor/Azure.Monitor.OpenTelemetry.Exporter/src/Azure.Monitor.OpenTelemetry.Exporter.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<Project Sdk="Microsoft.NET.Sdk">
<Project Sdk="Microsoft.NET.Sdk">
<PropertyGroup>
<Description>An OpenTelemetry .NET exporter that exports to Azure Monitor</Description>
<AssemblyTitle>AzureMonitor OpenTelemetry Exporter</AssemblyTitle>
Expand All @@ -9,6 +9,13 @@
<TargetFrameworks>$(RequiredTargetFrameworks)</TargetFrameworks>
<IncludeOperationsSharedSource>true</IncludeOperationsSharedSource>
<AotCompatOptOut>true</AotCompatOptOut>
<EnableConfigurationBindingGenerator>true</EnableConfigurationBindingGenerator>
Comment thread
eerhardt marked this conversation as resolved.
<!-- The configuration binding source generator emits SYSLIB1100/SYSLIB1101 for options
properties that cannot be bound from configuration (for example, the Credential
(TokenCredential) and Transport properties used by Azure.Core.ClientOptions). These
warnings are expected and safe to suppress since those properties are never set via
IConfiguration. -->
<NoWarn>$(NoWarn);SYSLIB1100;SYSLIB1101</NoWarn>
<DefineConstants>$(DefineConstants);AZURE_MONITOR_EXPORTER;</DefineConstants>
<IncludeAutorestDependency>true</IncludeAutorestDependency>
</PropertyGroup>
Expand Down
8 changes: 0 additions & 8 deletions sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/DefaultAzureMonitorExporterOptions.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
// Licensed under the MIT License.

using System;
using System.Diagnostics.CodeAnalysis;
using System.Globalization;
using Azure.Monitor.OpenTelemetry.Exporter.Internals.Diagnostics;
using Azure.Monitor.OpenTelemetry.Exporter.Internals.Platform;
Expand Down Expand Up @@ -128,12 +127,5 @@ private static void ConfigureSamplingOptions(string? samplerType, string? sample
AzureMonitorExporterEventSource.Log.ConfigureFailed(ex);
}
}

[UnconditionalSuppressMessage("Trimming", "IL2026", Justification = "Binding options is a known source of trim warnings; this is a deliberate usage.")]
[UnconditionalSuppressMessage("AOT", "IL3050", Justification = "Binding options is a known source of AOT warnings; this is a deliberate usage.")]
private static void BindIConfigurationOptions(IConfiguration configuration, AzureMonitorExporterOptions options)
{
configuration.GetSection(AzureMonitorExporterSectionFromConfig).Bind(options);
}
}
}
Loading