Migrate to Microsoft.Extensions.Logging (#1968)

Part 1/2 of #1908 - Migrate to M.E.Logging.
- Removed our ad-hoc logger setup. Use the default M.E.Logging setup
provided by M.E.Hosting instead.
- All classes/serivces now take `ILogger<T> logger` parameters and have
a private `_logger` field.

Everything is [Info] level right now. We can adjust later as we go.

TODO (before merging):
- [ ] Ensure Azure Pipelines logging directives still work
- [ ] Remove instances of protected loggers in favor of private instead

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: lbussell

src/ImageBuilder.Tests/AnnotateEolDigestsCommandTests.cs

@@ -11,10 +11,11 @@
 using Microsoft.DotNet.ImageBuilder.Models.Annotations;
 using Microsoft.DotNet.ImageBuilder.Models.Oci;
 using Microsoft.DotNet.ImageBuilder.Tests.Helpers;
+using Microsoft.Extensions.Logging;
 using Moq;
 using Newtonsoft.Json;
-using Xunit;
 using Xunit.Abstractions;
+using Xunit;
 
 namespace Microsoft.DotNet.ImageBuilder.Tests
 {
@@ -51,9 +52,9 @@ public async Task AnnotateEolDigestsCommand_AnnotationSuccess()
 
             Manifest manifest;
             lifecycleMetadataServiceMock.Verify(
-                o => o.AnnotateEolDigest("digest1", _globalDate, It.IsAny<ILoggerService>(), It.IsAny<bool>(), out manifest));
+                o => o.AnnotateEolDigest("digest1", _globalDate, It.IsAny<ILogger>(), It.IsAny<bool>(), out manifest));
             lifecycleMetadataServiceMock.Verify(
-                o => o.AnnotateEolDigest("digest2", _specificDigestDate, It.IsAny<ILoggerService>(), It.IsAny<bool>(), out manifest));
+                o => o.AnnotateEolDigest("digest2", _specificDigestDate, It.IsAny<ILogger>(), It.IsAny<bool>(), out manifest));
 
             string[] expectedAnnotationDigests =
                 [
@@ -106,7 +107,7 @@ public async Task AnnotateEolDigestsCommand_CheckAnnotations_AlreadyAnnotated_No
 
             Manifest manifest;
             lifecycleMetadataServiceMock.Verify(
-                o => o.AnnotateEolDigest(It.IsAny<string>(), It.IsAny<DateOnly>(), It.IsAny<ILoggerService>(), It.IsAny<bool>(), out manifest),
+                o => o.AnnotateEolDigest(It.IsAny<string>(), It.IsAny<DateOnly>(), It.IsAny<ILogger>(), It.IsAny<bool>(), out manifest),
                 Times.Never());
         }
 
@@ -128,7 +129,7 @@ public async Task AnnotateEolDigestsCommand_CheckAnnotations_AlreadyAnnotated_Ma
 
             Manifest manifest;
             lifecycleMetadataServiceMock.Verify(
-                o => o.AnnotateEolDigest(It.IsAny<string>(), It.IsAny<DateOnly>(), It.IsAny<ILoggerService>(), It.IsAny<bool>(), out manifest),
+                o => o.AnnotateEolDigest(It.IsAny<string>(), It.IsAny<DateOnly>(), It.IsAny<ILogger>(), It.IsAny<bool>(), out manifest),
                 Times.Never());
         }
 
@@ -152,7 +153,7 @@ private AnnotateEolDigestsCommand InitializeCommand(
             string eolDigestsListPath = Path.Combine(tempFolderContext.Path, "eol-digests.json");
             File.WriteAllText(eolDigestsListPath, JsonConvert.SerializeObject(eolAnnotations));
 
-            Mock<ILoggerService> loggerServiceMock = new();
+            Mock<ILogger<AnnotateEolDigestsCommand>> loggerServiceMock = new();
             lifecycleMetadataServiceMock = CreateLifecycleMetadataServiceMock(digestAlreadyAnnotated, digestAnnotationIsSuccessful, useNonMatchingDate);
             AnnotateEolDigestsCommand command = new(
                 loggerServiceMock.Object,
@@ -177,7 +178,7 @@ private Mock<ILifecycleMetadataService> CreateLifecycleMetadataServiceMock(bool
             };
 
             lifecycleMetadataServiceMock
-                .Setup(o => o.AnnotateEolDigest(It.Is<string>(digest => digest.Contains("digest1")), It.IsAny<DateOnly>(), It.IsAny<ILoggerService>(), It.IsAny<bool>(), out digest1Annotation))
+                .Setup(o => o.AnnotateEolDigest(It.Is<string>(digest => digest.Contains("digest1")), It.IsAny<DateOnly>(), It.IsAny<ILogger>(), It.IsAny<bool>(), out digest1Annotation))
                 .Returns(digestAnnotationIsSuccessful);
 
             Manifest digest2Annotation = new()
@@ -186,7 +187,7 @@ private Mock<ILifecycleMetadataService> CreateLifecycleMetadataServiceMock(bool
             };
 
             lifecycleMetadataServiceMock
-                .Setup(o => o.AnnotateEolDigest(It.Is<string>(digest => digest.Contains("digest2")), It.IsAny<DateOnly>(), It.IsAny<ILoggerService>(), It.IsAny<bool>(), out digest2Annotation))
+                .Setup(o => o.AnnotateEolDigest(It.Is<string>(digest => digest.Contains("digest2")), It.IsAny<DateOnly>(), It.IsAny<ILogger>(), It.IsAny<bool>(), out digest2Annotation))
                 .Returns(digestAnnotationIsSuccessful);
 
             return lifecycleMetadataServiceMock;
@@ -213,7 +214,7 @@ private static void SetupIsDigestAnnotatedForEolMethod(Mock<ILifecycleMetadataSe
             }
 
             lifecycleMetadataServiceMock
-                .Setup(o => o.IsDigestAnnotatedForEol(digest, It.IsAny<ILoggerService>(), It.IsAny<bool>(), out manifest))
+                .Setup(o => o.IsDigestAnnotatedForEol(digest, It.IsAny<ILogger>(), It.IsAny<bool>(), out manifest))
                 .Returns(digestAlreadyAnnotated);
         }
     }