Fix typo & use of service provider

Signed-off-by: Victor Chang <vicchang@nvidia.com>

Author: mocsharp

src/Plugins/MinIO/HealthCheckBuilder.cs

@@ -14,6 +14,7 @@
  * limitations under the License.
  */
 
+using System;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Diagnostics.HealthChecks;
 using Microsoft.Extensions.Logging;
@@ -28,17 +29,19 @@ public HealthCheckBuilder(string fullyQualifiedAssemblyName) : base(fullyQualifi
 
         public override IHealthChecksBuilder Configure(
             IHealthChecksBuilder builder,
-            IServiceProvider serviceProvider,
             HealthStatus? failureStatus = null,
             IEnumerable<string>? tags = null,
             TimeSpan? timeout = null)
         {
-            var minioClientFactory = serviceProvider.GetRequiredService<IMinIoClientFactory>();
-            var logger = serviceProvider.GetRequiredService<ILogger<MinIoHealthCheck>>();
 
             builder.Add(new HealthCheckRegistration(
                 ConfigurationKeys.StorageServiceName,
-                new MinIoHealthCheck(minioClientFactory, logger),
+                serviceProvider =>
+                {
+                    var minioClientFactory = serviceProvider.GetRequiredService<IMinIoClientFactory>();
+                    var logger = serviceProvider.GetRequiredService<ILogger<MinIoHealthCheck>>();
+                    return new MinIoHealthCheck(minioClientFactory, logger);
+                },
                 failureStatus,
                 tags,
                 timeout));

src/Storage/HealthCheckRegistrationBase.cs

@@ -49,7 +49,6 @@ private string ParseAssemblyName()
 
         public abstract IHealthChecksBuilder Configure(
             IHealthChecksBuilder builder,
-            IServiceProvider serviceProvider,
             HealthStatus? failureStatus = null,
             IEnumerable<string>? tags = null,
             TimeSpan? timeout = null);

src/Storage/IHealthChecksBuilderExtensions.cs

@@ -33,27 +33,24 @@ public static class IHealthChecksBuilderExtensions
         /// <param name="fullyQualifiedTypeName">Fully qualified type name of the service to use.</param>
         /// <returns>Instance of <see cref="IHealthChecksBuilder"/>.</returns>
         /// <exception cref="ConfigurationException"></exception>
-        public static IHealthChecksBuilder AddMOnaiDeployStorageHealthCheck(
+        public static IHealthChecksBuilder AddMonaiDeployStorageHealthCheck(
             this IHealthChecksBuilder builder,
-            IServiceProvider serviceProvider,
             string fullyQualifiedTypeName,
             HealthStatus? failureStatus = null,
             IEnumerable<string>? tags = null,
             TimeSpan? timeout = null)
-            => AddMOnaiDeployStorageHealthCheck(builder, serviceProvider, fullyQualifiedTypeName, new FileSystem(), failureStatus, tags, timeout);
+            => AddMonaiDeployStorageHealthCheck(builder, fullyQualifiedTypeName, new FileSystem(), failureStatus, tags, timeout);
 
         /// <summary>
         /// Configures health check for the MONAI Deploy Storage Service.
         /// </summary>
         /// <param name="builder">Instance of <see cref="IHealthChecksBuilder"/>.</param>
-        /// <param name="serviceProvider">Instance of <see cref="IServiceProvider"/>.</param>
         /// <param name="fullyQualifiedTypeName">Fully qualified type name of the service to use.</param>
         /// <param name="fileSystem">Instance of <see cref="IFileSystem"/>.</param>
         /// <returns>Instance of <see cref="IHealthChecksBuilder"/>.</returns>
         /// <exception cref="ConfigurationException"></exception>
-        public static IHealthChecksBuilder AddMOnaiDeployStorageHealthCheck(
+        public static IHealthChecksBuilder AddMonaiDeployStorageHealthCheck(
             this IHealthChecksBuilder builder,
-            IServiceProvider serviceProvider,
             string fullyQualifiedTypeName,
             IFileSystem fileSystem,
             HealthStatus? failureStatus = null,
@@ -83,7 +80,7 @@ public static IHealthChecksBuilder AddMOnaiDeployStorageHealthCheck(
                 throw new ConfigurationException($"The configured type '{fullyQualifiedTypeName}' does not implement the {typeof(IStorageService).Name} interface.");
             }
 
-            healthCheckBuilder.Configure(builder, serviceProvider, failureStatus, tags, timeout);
+            healthCheckBuilder.Configure(builder, failureStatus, tags, timeout);
 
             AppDomain.CurrentDomain.AssemblyResolve -= resolveEventHandler;
             return builder;