Skip to content

Do not fail if single cluster apply fails#130

Merged
sdseaton merged 1 commit intomainfrom
bugfix/cluster_apply
Dec 17, 2025
Merged

Do not fail if single cluster apply fails#130
sdseaton merged 1 commit intomainfrom
bugfix/cluster_apply

Conversation

@sdseaton
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings December 17, 2025 16:24
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR enhances the fault tolerance of the cluster orchestration process by adding error handling to prevent a single cluster failure from stopping the entire multi-cluster apply operation. When one cluster fails, the system now logs the error and continues processing remaining clusters, ensuring maximum coverage of change deployments.

Key Changes:

  • Wrapped the cluster apply logic in a try-catch block to isolate failures
  • Added CreateFailedClusterResult helper method to generate standardized error results
  • Ensured all results (both successful and failed) are included in the return value

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread KustoSchemaTools/KustoClusterOrchestrator.cs
Comment thread KustoSchemaTools/KustoClusterOrchestrator.cs
@sdseaton sdseaton merged commit 67847e8 into main Dec 17, 2025
11 checks passed
@sdseaton sdseaton deleted the bugfix/cluster_apply branch December 17, 2025 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants