Adding logging for Storage operations (#7471)

Author: karshinlin

src/WebJobs.Script/ScriptHostBuilderExtensions.cs

@@ -455,16 +455,19 @@ private static IDistributedLockManager GetBlobLockManager(IServiceProvider provi
         {
             var azureStorageProvider = provider.GetRequiredService<IAzureStorageProvider>();
             var loggerFactory = provider.GetRequiredService<ILoggerFactory>();
+            var logger = loggerFactory.CreateLogger<IDistributedLockManager>();
             try
             {
                 var container = azureStorageProvider.GetBlobContainerClient();
+                logger.LogDebug("Using BlobLeaseDistributedLockManager in Functions Host.");
                 return new BlobLeaseDistributedLockManager(loggerFactory, azureStorageProvider);
             }
             catch (InvalidOperationException)
             {
                 // If there is an error getting the container client,
                 // register an InMemoryDistributedLockManager.
                 // This signals a failed validation in connection configuration (i.e. could not create the storage client).
+                logger.LogDebug("Using InMemoryDistributedLockManager in Functions Host.");
                 return new InMemoryDistributedLockManager();
             }
         }

src/WebJobs.Script/StorageProvider/BlobServiceClientProvider.cs

@@ -16,7 +16,7 @@ namespace Microsoft.Azure.WebJobs.Script.StorageProvider
     /// </summary>
     internal class BlobServiceClientProvider : StorageClientProvider<BlobServiceClient, BlobClientOptions>
     {
-        public BlobServiceClientProvider(IConfiguration configuration, AzureComponentFactory componentFactory, AzureEventSourceLogForwarder logForwarder, ILogger<BlobServiceClient> logger)
+        public BlobServiceClientProvider(IConfiguration configuration, AzureComponentFactory componentFactory, AzureEventSourceLogForwarder logForwarder, ILogger<BlobServiceClientProvider> logger)
             : base(configuration, componentFactory, logForwarder, logger) { }
 
         /// <inheritdoc/>

src/WebJobs.Script/StorageProvider/StorageClientProvider.cs

@@ -32,7 +32,7 @@ internal abstract class StorageClientProvider<TClient, TClientOptions> where TCl
         /// <param name="configuration">The configuration to use when creating Client-specific objects. <see cref="IConfiguration"/></param>
         /// <param name="componentFactory">The Azure factory responsible for creating clients. <see cref="AzureComponentFactory"/></param>
         /// <param name="logForwarder">Log forwarder that forwards events to ILogger. <see cref="AzureEventSourceLogForwarder"/></param>
-        public StorageClientProvider(IConfiguration configuration, AzureComponentFactory componentFactory, AzureEventSourceLogForwarder logForwarder, ILogger<TClient> logger)
+        public StorageClientProvider(IConfiguration configuration, AzureComponentFactory componentFactory, AzureEventSourceLogForwarder logForwarder, ILogger<StorageClientProvider<TClient, TClientOptions>> logger)
         {
             _configuration = configuration;
             _componentFactory = componentFactory;
@@ -65,7 +65,7 @@ public virtual bool TryGet(string name, out TClient client)
             catch (Exception ex)
             {
                 client = default(TClient);
-                _logger.LogError(ex, "Could not create Storage Client");
+                _logger.LogDebug("Could not create Storage Client. Exception: {0}", ex.ToString());
                 return false;
             }
         }
@@ -86,7 +86,7 @@ public virtual bool TryGetFromConnectionString(string connectionString, out TCli
             catch (Exception ex)
             {
                 client = default(TClient);
-                _logger.LogError(ex, "Could not create Storage Client from a connection string.");
+                _logger.LogDebug("Could not create Storage Client from a connection string. Exception: {0}", ex.ToString());
                 return false;
             }
         }
@@ -190,7 +190,7 @@ protected virtual bool TryGetServiceUri(IConfiguration configuration, out Uri se
             }
             catch (Exception ex)
             {
-                _logger.LogError(ex, "Could not parse serviceUri from the configuration.");
+                _logger.LogDebug("Could not parse serviceUri from the configuration. Exception: {0}", ex.ToString());
             }
 
             serviceUri = default(Uri);