remove all tracing related code

Author: zhiyuanliang-ms

Aspire.sln

@@ -661,16 +661,11 @@ Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Aspire.Azure.Npgsql.EntityF
 EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Aspire.Components.Common.Tests", "tests\Aspire.Components.Common.Tests\Aspire.Components.Common.Tests.csproj", "{30950CEB-2232-F9FC-04FF-ADDCB8AC30A7}"
 EndProject
-<<<<<<< HEAD
-<<<<<<< HEAD
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Aspire.Azure.AppConfiguration", "src\Components\Aspire.Azure.AppConfiguration\Aspire.Azure.AppConfiguration.csproj", "{C33CE874-27B7-4194-A2E7-D0CD950997CC}"
 EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Aspire.Azure.AppConfiguration.Tests", "tests\Aspire.Azure.AppConfiguration.Tests\Aspire.Azure.AppConfiguration.Tests.csproj", "{FE972BF3-F448-4CF0-8544-0056B26BF006}"
-=======
 Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "src", "src", "{827E0CD3-B72D-47B6-A68D-7590B98EB39B}"
 EndProject
-=======
->>>>>>> 0e182c9de0132af77787fdcc550e19b63a50e5e1
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Aspire.Hosting.Azure.ContainerRegistry", "src\Aspire.Hosting.Azure.ContainerRegistry\Aspire.Hosting.Azure.ContainerRegistry.csproj", "{6CBA29C8-FF78-4ABC-BEFA-2A53CB4DB2A3}"
 EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Aspire.Hosting.Azure.AppService", "src\Aspire.Hosting.Azure.AppService\Aspire.Hosting.Azure.AppService.csproj", "{5DDF8E89-FBBD-4A6F-BF32-7D2140724941}"
@@ -680,7 +675,6 @@ EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "AzureAppService.ApiService", "playground\AzureAppService\AzureAppService.ApiService\AzureAppService.ApiService.csproj", "{A617DC84-65DA-41B5-B378-6C2F569CEE48}"
 EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "AzureAppService.AppHost", "playground\AzureAppService\AzureAppService.AppHost\AzureAppService.AppHost.csproj", "{2C879943-DF34-44FA-B2C3-29D97F24DD76}"
->>>>>>> aad850bcfea054da216d79cdf54b049740daab95
 EndProject
 Global
 	GlobalSection(SolutionConfigurationPlatforms) = preSolution

src/Components/Aspire.Azure.AppConfiguration/Aspire.Azure.AppConfiguration.csproj

@@ -22,6 +22,7 @@
   <ItemGroup>
     <PackageReference Include="Microsoft.Extensions.Azure" />
     <PackageReference Include="Microsoft.Extensions.Configuration.Binder" />
+    <PackageReference Include="Microsoft.Extensions.Diagnostics.HealthChecks" />
     <PackageReference Include="Microsoft.Extensions.Hosting.Abstractions" />
     <PackageReference Include="OpenTelemetry.Extensions.Hosting" />
     <PackageReference Include="Microsoft.Extensions.Configuration.AzureAppConfiguration" />

src/Components/Aspire.Azure.AppConfiguration/AspireAppConfigurationExtensions.cs

@@ -64,13 +64,7 @@ public static void AddAzureAppConfiguration(
 
         builder.Services.AddAzureAppConfiguration(); // register IConfigurationRefresherProvider service
 
-        if (!settings.DisableTracing)
-        {
-            // WIP: https://github.com/Azure/AppConfiguration-DotnetProvider/pull/645
-            // Will be supported in the next 8.2.0 release
-            builder.Services.AddOpenTelemetry()
-                .WithTracing(traceBuilder =>
-                    traceBuilder.AddSource(["Microsoft.Extensions.Configuration.AzureAppConfiguration"]));
-        }
+        // WIP: https://github.com/Azure/AppConfiguration-DotnetProvider/pull/645
+        // Tracing will be supported in the next 8.2.0 release
     }
 }

src/Components/Aspire.Azure.AppConfiguration/AzureAppConfigurationSettings.cs

@@ -22,14 +22,6 @@ public sealed class AzureAppConfigurationSettings : IConnectionStringSettings
     /// </summary>
     public TokenCredential? Credential { 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) &&

src/Components/Aspire.Azure.AppConfiguration/ConfigurationSchema.json

@@ -19,11 +19,6 @@
             "AppConfiguration": {
               "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",

tests/Aspire.Azure.AppConfiguration.Tests/ConfigurationTests.cs

@@ -11,7 +11,6 @@ public class ConfigurationTests
     public void EndpointUriIsNullByDefault()
         => Assert.Null(new AzureAppConfigurationSettings().Endpoint);
 
-    [Fact]
-    public void TracingIsEnabledByDefault()
-        => Assert.False(new AzureAppConfigurationSettings().DisableTracing);
+    // WIP: https://github.com/Azure/AppConfiguration-DotnetProvider/pull/645
+    // Tracing will be supported in the next 8.2.0 release
 }

tests/Aspire.Azure.AppConfiguration.Tests/ConformanceTests.cs

@@ -32,12 +32,8 @@ public class ConformanceTests : ConformanceTests<IConfigurationRefresherProvider
         {
           "Aspire": {
             "Azure": {
-              "Data": {
-                "AppConfiguration": {
-                  "Endpoint": "http://YOUR_URI",
-                  "DisableHealthChecks": true,
-                  "DisableTracing": false
-                }
+              "AppConfiguration": {
+                "Endpoint": "http://YOUR_URI"
               }
             }
           }
@@ -81,8 +77,7 @@ protected override void RegisterComponent(HostApplicationBuilder builder, Action
 
     protected override (string json, string error)[] InvalidJsonToErrorMessage => new[]
         {
-            ("""{"Aspire": { "Azure": { "AppConfiguration": { "Endpoint": "YOUR_URI"}}}}""", "Value does not match format \"uri\""),
-            ("""{"Aspire": { "Azure": { "AppConfiguration": { "Endpoint": "http://YOUR_URI", "DisableTracing": "true"}}}}""", "Value is \"string\" but should be \"boolean\""),
+            ("""{"Aspire": { "Azure": { "AppConfiguration": { "Endpoint": "YOUR_URI"}}}}""", "Value does not match format \"uri\"")
         };
 
     protected override void SetHealthCheck(AzureAppConfigurationSettings options, bool enabled)
@@ -98,10 +93,9 @@ protected override void SetTracing(AzureAppConfigurationSettings options, bool e
         => throw new NotImplementedException();
 
     protected override void TriggerActivity(IConfigurationRefresherProvider service)
-    {
-        Thread.Sleep(1000);
-        service.Refreshers.First().RefreshAsync().ConfigureAwait(false).GetAwaiter().GetResult();
-    }
+        // WIP: https://github.com/Azure/AppConfiguration-DotnetProvider/pull/645
+        // Will be supported in the next 8.2.0 release
+        => throw new NotImplementedException();
 
     [Fact]
     public void TracingEnablesTheRightActivitySource()