Skip to content

Add tracing support for Azure App Configuration component #9323

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
merged 5 commits into from
May 15, 2025
Merged

Add tracing support for Azure App Configuration component #9323

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
0124828
add tracing support for Azure App Configuration component
zhiyuanliang-ms May 15, 2025
47d06ca
avoid using Thread.Sleep
zhiyuanliang-ms May 15, 2025
1d61a74
update progress
zhiyuanliang-ms May 15, 2025
2ddd37f
Remove unnecessary dependency
eerhardt May 15, 2025
29e39e1
Update readme
sebastienros May 15, 2025
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
4 changes: 2 additions & 2 deletions Directory.Packages.props
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,8 @@
<PackageVersion Include="Azure.Security.KeyVault.Keys" Version="4.7.0" />
<PackageVersion Include="Azure.Storage.Blobs" Version="12.24.0" />
<PackageVersion Include="Azure.Storage.Queues" Version="12.22.0" />
<PackageVersion Include="Microsoft.Extensions.Configuration.AzureAppConfiguration" Version="8.1.2" />
<PackageVersion Include="Microsoft.Azure.AppConfiguration.AspNetCore" Version="8.1.2" />
<PackageVersion Include="Microsoft.Extensions.Configuration.AzureAppConfiguration" Version="8.2.0" />
<PackageVersion Include="Microsoft.Azure.AppConfiguration.AspNetCore" Version="8.2.0" />
<PackageVersion Include="Microsoft.Azure.Cosmos" Version="3.49.0" />
<PackageVersion Include="Microsoft.Azure.SignalR" Version="1.30.3" />
<PackageVersion Include="Microsoft.Azure.SignalR.Management" Version="1.30.3" />
Expand Down
7 changes: 7 additions & 0 deletions ...rosoft.Extensions.Configuration.AzureAppConfiguration/AspireAppConfigurationExtensions.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,5 +61,12 @@ public static void AddAzureAppConfiguration(
settings.Optional);

builder.Services.AddAzureAppConfiguration(); // register IConfigurationRefresherProvider service

if (!settings.DisableTracing)
{
builder.Services.AddOpenTelemetry()
.WithTracing(traceBuilder =>
traceBuilder.AddSource(["Microsoft.Extensions.Configuration.AzureAppConfiguration"]));
}
}
}
8 changes: 8 additions & 0 deletions ...Microsoft.Extensions.Configuration.AzureAppConfiguration/AzureAppConfigurationSettings.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,14 @@ public sealed class AzureAppConfigurationSettings : IConnectionStringSettings
/// </summary>
public bool Optional { get; set; }

/// <summary>
/// Gets or sets a boolean value that indicates whether the OpenTelemetry tracing is disabled or not.
/// </summary>
/// <value>
/// The default value is <see langword="false"/>.
/// </value>
public bool DisableTracing { get; set; }

void IConnectionStringSettings.ParseConnectionString(string? connectionString)
{
if (!string.IsNullOrEmpty(connectionString) &&
Expand Down
5 changes: 5 additions & 0 deletions .../Aspire.Microsoft.Extensions.Configuration.AzureAppConfiguration/ConfigurationSchema.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,11 @@
"AzureAppConfiguration": {
"type": "object",
"properties": {
"DisableTracing": {
"type": "boolean",
"description": "Gets or sets a boolean value that indicates whether the OpenTelemetry tracing is disabled or not.",
"default": false
},
"Endpoint": {
"type": "string",
"format": "uri",
Expand Down
5 changes: 3 additions & 2 deletions ...pire.Microsoft.Extensions.Configuration.AzureAppConfiguration.Tests/ConfigurationTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ public class ConfigurationTests
public void EndpointUriIsNullByDefault()
=> Assert.Null(new AzureAppConfigurationSettings().Endpoint);

// WIP: https://github.com/Azure/AppConfiguration-DotnetProvider/pull/645
// Tracing will be supported in the next 8.2.0 release
[Fact]
public void TracingIsEnabledByDefault()
=> Assert.False(new AzureAppConfigurationSettings().DisableTracing);
}
20 changes: 10 additions & 10 deletions ...Aspire.Microsoft.Extensions.Configuration.AzureAppConfiguration.Tests/ConformanceTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,8 @@ public class ConformanceTests : ConformanceTests<IConfigurationRefresherProvider
"Azure": {
"AppConfiguration": {
"Endpoint": "http://YOUR_URI",
"Optional": true
"Optional": true,
"DisableTracing": false
}
}
}
Expand Down Expand Up @@ -78,7 +79,8 @@ protected override void RegisterComponent(HostApplicationBuilder builder, Action
protected override (string json, string error)[] InvalidJsonToErrorMessage => new[]
{
("""{"Aspire": { "Microsoft": { "Extensions": { "Configuration": { "AzureAppConfiguration": { "Endpoint": "YOUR_URI"}}}}}}""", "Value does not match format \"uri\""),
("""{"Aspire": { "Microsoft": { "Extensions": { "Configuration": { "AzureAppConfiguration": { "Endpoint": "http://YOUR_URI", "Optional": "true"}}}}}}""", "Value is \"string\" but should be \"boolean\"")
("""{"Aspire": { "Microsoft": { "Extensions": { "Configuration": { "AzureAppConfiguration": { "Endpoint": "http://YOUR_URI", "Optional": "true"}}}}}}""", "Value is \"string\" but should be \"boolean\""),
("""{"Aspire": { "Microsoft": { "Extensions": { "Configuration": { "AzureAppConfiguration": { "Endpoint": "http://YOUR_URI", "DisableTracing": "true"}}}}}}""", "Value is \"string\" but should be \"boolean\""),
};

protected override void SetHealthCheck(AzureAppConfigurationSettings options, bool enabled)
Expand All @@ -89,19 +91,17 @@ protected override void SetMetrics(AzureAppConfigurationSettings options, bool e
=> throw new NotImplementedException();

protected override void SetTracing(AzureAppConfigurationSettings options, bool enabled)
// WIP: https://github.com/Azure/AppConfiguration-DotnetProvider/pull/645
// Will be supported in the next 8.2.0 release
=> throw new NotImplementedException();
=> options.DisableTracing = !enabled;

protected override void TriggerActivity(IConfigurationRefresherProvider service)
// WIP: https://github.com/Azure/AppConfiguration-DotnetProvider/pull/645
// Will be supported in the next 8.2.0 release
=> throw new NotImplementedException();
{
Thread.Sleep(1000);
service.Refreshers.First().RefreshAsync().ConfigureAwait(false).GetAwaiter().GetResult();
}

[Fact]
public void TracingEnablesTheRightActivitySource()
// WIP: Waiting for App Configuration Provider 8.2.0 release
=> RemoteExecutor.Invoke(() => /*ActivitySourceTest(key: null)*/ null).Dispose();
=> RemoteExecutor.Invoke(() => ActivitySourceTest(key: null)).Dispose();

internal sealed class EmptyTokenCredential : TokenCredential
{
Expand Down
Loading