Remove omexlogger

src/Logging/Internal/EventSource/EmptyServiceContext.cs → src/Abstractions/ServiceContext/EmptyServiceContext.cs

@@ -3,15 +3,21 @@
 
 using System;
 
-namespace Microsoft.Omex.Extensions.Logging
+namespace Microsoft.Omex.Extensions.Abstractions.ServiceContext
 {
 	/// <summary>
 	/// IServiceContext without any information
 	/// </summary>
-	internal class EmptyServiceContext : IServiceContext
+	public class EmptyServiceContext : IServiceContext
 	{
+		/// <summary>
+		/// Service fabric PartitionId
+		/// </summary>
 		public Guid PartitionId => Guid.Empty;
 
+		/// <summary>
+		/// Service fabric
+		/// </summary>
 		public long ReplicaOrInstanceId => 0;
 	}
 }

src/Logging/IServiceContext.cs → src/Abstractions/ServiceContext/IServiceContext.cs

@@ -3,7 +3,7 @@
 
 using System;
 
-namespace Microsoft.Omex.Extensions.Logging
+namespace Microsoft.Omex.Extensions.Abstractions.ServiceContext
 {
 	/// <summary>
 	/// Interface with service context information

src/Hosting.Services/HostBuilderExtensions.cs

@@ -8,9 +8,11 @@
 using Microsoft.Extensions.DependencyInjection.Extensions;
 using Microsoft.Extensions.Hosting;
 using Microsoft.Extensions.Logging;
+using Microsoft.Extensions.Logging.Configuration;
 using Microsoft.Omex.Extensions.Abstractions;
 using Microsoft.Omex.Extensions.Abstractions.Accessors;
 using Microsoft.Omex.Extensions.Abstractions.ExecutionContext;
+using Microsoft.Omex.Extensions.Abstractions.ServiceContext;
 using Microsoft.Omex.Extensions.Logging;
 using Microsoft.ServiceFabric.Data;
 
@@ -75,53 +77,39 @@ private static IHost BuildServiceFabricService<TRunner, TService, TContext>(
 		{
 			string serviceNameForLogging = serviceName;
 
-			try
+			if (string.IsNullOrWhiteSpace(serviceName))
 			{
-				if (string.IsNullOrWhiteSpace(serviceName))
-				{
-					// use executing assembly name for logging since application name not available
-					serviceNameForLogging = Assembly.GetExecutingAssembly().GetName().FullName;
-					throw new ArgumentException("Service type name is null of whitespace", nameof(serviceName));
-				}
-
-				builderAction(new ServiceFabricHostBuilder<TService, TContext>(builder));
-
-				IHost host = builder
-					.ConfigureServices((context, collection) =>
-					{
-						collection
-							.Configure<ServiceRegistratorOptions>(options =>
-							{
-								options.ServiceTypeName = serviceName;
-							})
-							.AddOmexServiceFabricDependencies<TContext>()
-							.AddSingleton<IOmexServiceRegistrator, TRunner>()
-							.AddHostedService<OmexHostedService>();
-					})
-					.UseDefaultServiceProvider(options =>
-					{
-						options.ValidateOnBuild = true;
-						options.ValidateScopes = true;
-					})
-#pragma warning disable OMEX188 // AddOmexLogging using OmexLogger is obsolete. DiagnosticId = "OMEX188"
-					.ConfigureLogging(builder => builder.AddOmexLogging())
-#pragma warning restore OMEX188 // AddOmexLogging using OmexLogger is obsolete. DiagnosticId = "OMEX188"
-					.Build();
+				// use executing assembly name for logging since application name not available
+				serviceNameForLogging = Assembly.GetExecutingAssembly().GetName().FullName;
+				throw new ArgumentException("Service type name is null of whitespace", nameof(serviceName));
+			}
 
-#pragma warning disable OMEX188 // InitializationLogger using OmexLogger is obsolete. DiagnosticId = "OMEX188"
-				InitializationLogger.LogInitializationSucceed(serviceNameForLogging);
-#pragma warning restore OMEX188 // InitializationLogger using OmexLogger is obsolete. DiagnosticId = "OMEX188"
+			builderAction(new ServiceFabricHostBuilder<TService, TContext>(builder));
 
-				return host;
-			}
-			catch (Exception e)
-			{
-#pragma warning disable OMEX188 // InitializationLogger using OmexLogger is obsolete. DiagnosticId = "OMEX188"
-				InitializationLogger.LogInitializationFail(serviceNameForLogging, e);
-#pragma warning restore OMEX188 // InitializationLogger using OmexLogger is obsolete. DiagnosticId = "OMEX188"
+			IHost host = builder
+				.ConfigureServices((context, collection) =>
+				{
+					collection
+						.Configure<ServiceRegistratorOptions>(options =>
+						{
+							options.ServiceTypeName = serviceName;
+						})
+						.AddOmexServiceFabricDependencies<TContext>()
+						.AddSingleton<IOmexServiceRegistrator, TRunner>()
+						.AddHostedService<OmexHostedService>();
+					// If specific context constructs are not registered, add defaults for telemetry purposes
+					collection.TryAddTransient<IServiceContext, EmptyServiceContext>();
+					collection.TryAddTransient<IExecutionContext, BaseExecutionContext>();
+				})
+				.UseDefaultServiceProvider(options =>
+				{
+					options.ValidateOnBuild = true;
+					options.ValidateScopes = true;
+				})
+				.ConfigureLogging(builder => builder.AddConfiguration())
+				.Build();
 
-				throw;
-			}
+			return host;
 		}
 
 		internal static IServiceCollection TryAddAccessor<TValue, TBase>(this IServiceCollection collection)

src/Hosting.Services/Internal/OmexServiceFabricContext.cs

@@ -4,7 +4,7 @@
 using System;
 using System.Fabric;
 using Microsoft.Omex.Extensions.Abstractions;
-using Microsoft.Omex.Extensions.Logging;
+using Microsoft.Omex.Extensions.Abstractions.ServiceContext;
 
 namespace Microsoft.Omex.Extensions.Hosting.Services
 {

src/Hosting/Certificates/CertificateReader.cs

@@ -14,17 +14,11 @@ namespace Microsoft.Omex.Extensions.Hosting.Certificates
 	internal class CertificateReader : ICertificateReader
 	{
 		private readonly ICertificateStore m_certificateStore;
-		private readonly ILogger m_logger;
 		private readonly ConcurrentDictionary<StoreName, CertificateInformation[]> m_certificatesCache;
 
-		public CertificateReader(ICertificateStore certificateStore, ILogger<CertificateReader> logger) : this(certificateStore, logger as ILogger)
-		{
-		}
-
-		internal CertificateReader(ICertificateStore certificateStore, ILogger logger)
+		public CertificateReader(ICertificateStore certificateStore)
 		{
 			m_certificateStore = certificateStore;
-			m_logger = logger;
 			m_certificatesCache = new ConcurrentDictionary<StoreName, CertificateInformation[]>();
 		}
 
@@ -42,40 +36,20 @@ internal CertificateReader(ICertificateStore certificateStore, ILogger logger)
 				}
 			}
 
-			if (matchingCertificate == null)
-			{
-				m_logger.LogError(Tag.Create(), "Could not find certificate with {0} thumbprint.", thumbprint);
-			}
-			else
-			{
-				m_logger.LogDebug(Tag.Create(), "Found certificate with {0} thumbprint.", thumbprint);
-			}
-
 			return matchingCertificate;
 		}
 
 		public IEnumerable<X509Certificate2> GetCertificatesByCommonName(string commonName, bool refreshCache, StoreName storeName)
 		{
 			Validation.ThrowIfNullOrWhiteSpace(commonName, nameof(commonName));
 
-			bool certFound = false;
 			foreach (CertificateInformation info in LoadCertificates(storeName, refreshCache))
 			{
 				if (string.Equals(info.CommonName, commonName, StringComparison.Ordinal))
 				{
-					certFound = true;
 					yield return info.Certificate;
 				}
 			}
-
-			if (certFound)
-			{
-				m_logger.LogDebug(Tag.Create(), "Found at least one certificate with {0} as common name.", commonName);
-			}
-			else
-			{
-				m_logger.LogError(Tag.Create(), "Could not find certificate with {0} as common name.", commonName);
-			}
 		}
 
 		public X509Certificate2? GetCertificateByCommonName(string commonName, bool refreshCache, StoreName storeName) =>
@@ -86,12 +60,7 @@ private CertificateInformation[] LoadCertificates(StoreName storeName, bool refr
 			CertificateInformation[]? certificates;
 			if (refreshCache || !m_certificatesCache.TryGetValue(storeName, out certificates))
 			{
-				m_logger.LogInformation(Tag.Create(), "Updating certificates cache for store '{0}'.", storeName);
-
 				certificates = m_certificateStore.GetAllCertificates(storeName, StoreLocation.LocalMachine).ToArray();
-
-				m_logger.LogInformation(Tag.Create(), "Inserting '{0}' certificates to certificates cache for store '{1}'.", certificates.Length, storeName);
-
 				m_certificatesCache[storeName] = certificates;
 			}
 

src/Hosting/Certificates/InitializationCertificateReader.cs

@@ -17,8 +17,6 @@ public static class InitializationCertificateReader
 		/// <summary>
 		/// Instance of CertificateReader
 		/// </summary>
-#pragma warning disable OMEX188 // InitializationLogger using OmexLogger is obsolete and is pending for removal by 1 July 2024.
-		public static ICertificateReader Instance { get; } = new CertificateReader(new CertificateStore(), InitializationLogger.Instance);
-#pragma warning restore OMEX188 // InitializationLogger using OmexLogger is obsolete and is pending for removal by 1 July 2024.
+		public static ICertificateReader Instance { get; } = new CertificateReader(new CertificateStore());
 	}
 }

src/Hosting/ServiceCollectionExtensions.cs

@@ -1,12 +1,10 @@
 // Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT license.
 
-using System;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.DependencyInjection.Extensions;
 using Microsoft.Extensions.Hosting;
 using Microsoft.Omex.Extensions.Hosting.Certificates;
-using Microsoft.Omex.Extensions.Logging;
 
 namespace Microsoft.Omex.Extensions.Hosting
 {
@@ -26,11 +24,7 @@ public static IHostBuilder AddOmexServices(this IHostBuilder builder) =>
 		/// Add Omex Logging and ActivitySource dependencies
 		/// </summary>
 		public static IServiceCollection AddOmexServices(this IServiceCollection collection) =>
-#pragma warning disable OMEX188 // OmexLogger and OmexLogEventSource are obsolete and pending for removal by 1 July 2024. Please consider using a different Logger.
-			collection
-				.AddOmexLogging()
-#pragma warning restore OMEX188 // OmexLogger and OmexLogEventSource are obsolete and pending for removal by 1 July 2024. Please consider using a different Logger.
-				.AddOmexActivitySource();
+			collection.AddOmexActivitySource();
 
 		/// <summary>
 		/// Add Omex Logging and ActivitySource dependencies