Merge pull request #130 from github/bugfix/cluster_apply

Do not fail if single cluster apply fails

Author: sdseaton

KustoSchemaTools/KustoClusterOrchestrator.cs

@@ -96,14 +96,34 @@ public async Task<List<ScriptExecuteCommandResult>> ApplyAsync(string clusterCon
                 var clusterUrl = changeSet.To.Url;
 
                 var kustoHandler = KustoClusterHandlerFactory.Create(clusterName, clusterUrl);
-                var result = await kustoHandler.WriteAsync(changeSet);
-                
-                // Add the results from this cluster to the overall results list
-                allResults.AddRange(result);
+                try
+                {
+                    var result = await kustoHandler.WriteAsync(changeSet);
+
+                    // Add the results from this cluster to the overall results list
+                    allResults.AddRange(result);
+                }
+                catch (Exception ex)
+                {
+                    Log.LogError(ex, $"Failed to apply changes to cluster: {clusterName}");
+                    allResults.Add(CreateFailedClusterResult(clusterName, clusterUrl, ex));
+                }
             }
 
             Log.LogInformation($"Finished applying. Total scripts executed: {allResults.Count}");
             return allResults;
         }
+
+        private static ScriptExecuteCommandResult CreateFailedClusterResult(string clusterName, string clusterUrl, Exception exception)
+        {
+            return new ScriptExecuteCommandResult
+            {
+                CommandType = $"cluster-script:{clusterName}",
+                OperationId = Guid.Empty,
+                Result = "Failed",
+                Reason = $"{exception.GetType().Name}: {exception.Message}",
+                CommandText = $"Apply cluster '{clusterName}' ({clusterUrl})"
+            };
+        }
     }
 }