Removes nlog in favour of appinsights telemetry

src/ApprenticeFeedback/SFA.DAS.ApprenticeFeedback.Api/Controllers/ApprenticeFeedbackTargetController.cs

@@ -34,6 +34,7 @@ public async Task<IActionResult> GetFeedbackTargetsForUpdate(int batchSize)
         {
             try
             {
+                _logger.LogInformation("Getting feedback targets for update");
                 var result = await _mediator.Send(new GetFeedbackTargetsForUpdateQuery() { BatchSize = batchSize });
                 return Ok(result.FeedbackTargetsForUpdate);
             }

src/ApprenticeFeedback/SFA.DAS.ApprenticeFeedback.Api/Program.cs

@@ -1,6 +1,5 @@
 using Microsoft.AspNetCore.Hosting;
 using Microsoft.Extensions.Hosting;
-using NLog.Web;
 
 namespace SFA.DAS.ApprenticeFeedback.Api
 {
@@ -16,7 +15,6 @@ public static IHostBuilder CreateHostBuilder(string[] args) =>
                 .ConfigureWebHostDefaults(webBuilder =>
                 {
                     webBuilder.UseStartup<Startup>();
-                })
-                .UseNLog();
+                });
     }
 }

src/ApprenticeFeedback/SFA.DAS.ApprenticeFeedback.Api/SFA.DAS.ApprenticeFeedback.Api.csproj

@@ -5,27 +5,9 @@
     <NoWin32Manifest>true</NoWin32Manifest>
   </PropertyGroup>
 
-  <ItemGroup>
-    <Content Remove="nlog.Debug.config" />
-    <Content Remove="nlog.Release.config" />
-  </ItemGroup>
-
-  <ItemGroup>
-    <None Include="nlog.Debug.config">
-      <DependentUpon>nlog.config</DependentUpon>
-      <IsTransformFile>true</IsTransformFile>
-    </None>
-    <None Include="nlog.Release.config">
-      <DependentUpon>nlog.config</DependentUpon>
-      <IsTransformFile>true</IsTransformFile>
-    </None>
-  </ItemGroup>
-
   <ItemGroup>
     <PackageReference Include="MediatR" />
     <PackageReference Include="Microsoft.ApplicationInsights.AspNetCore" />
-    <PackageReference Include="NLog.Web.AspNetCore" />
-    <PackageReference Include="SFA.DAS.NLog.Targets.Redis" />
     <PackageReference Include="Swashbuckle.AspNetCore" />
   </ItemGroup>
 
@@ -34,12 +16,4 @@
     <ProjectReference Include="..\SFA.DAS.ApprenticeFeedback\SFA.DAS.ApprenticeFeedback.csproj" />
   </ItemGroup>
 
-  <ItemGroup>
-    <Content Update="nlog.config">
-      <TransformOnBuild>true</TransformOnBuild>
-    </Content>
-  </ItemGroup>
-
-
-
 </Project>

src/ApprenticeFeedback/SFA.DAS.ApprenticeFeedback.Api/Startup.cs

@@ -1,4 +1,3 @@
-using MediatR;
 using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Hosting;
 using Microsoft.AspNetCore.Mvc.Authorization;
@@ -14,6 +13,7 @@
 using SFA.DAS.SharedOuterApi.AppStart;
 using SFA.DAS.SharedOuterApi.Infrastructure.HealthCheck;
 using System.Collections.Generic;
+using Microsoft.Extensions.Logging;
 using SFA.DAS.SharedOuterApi.Apprentice.GovUK.Auth.Application.Commands;
 
 
@@ -77,7 +77,7 @@ public void ConfigureServices(IServiceCollection services)
                 options.LowercaseQueryStrings = true;
             }).AddMvc();
 
-            services.AddApplicationInsightsTelemetry(_configuration["APPINSIGHTS_INSTRUMENTATIONKEY"]);
+            services.AddOpenTelemetryRegistration(_configuration["APPLICATIONINSIGHTS_CONNECTION_STRING"]!);
 
             services.AddSwaggerGen(c =>
             {

src/ApprenticeFeedback/SFA.DAS.ApprenticeFeedback.Api/appsettings.json

@@ -11,5 +11,5 @@
   "ConfigNames": "SFA.DAS.ApprenticeFeedback.OuterApi",
   "Environment": "LOCAL",
   "Version": "1.0",
-  "APPINSIGHTS_INSTRUMENTATIONKEY": ""
+  "APPLICATIONINSIGHTS_CONNECTION_STRING": ""
 }