Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Aug 18, 2025

This PR implements a complete Azure Synapse health check package following the established patterns in the repository.

Features

  • Azure Synapse workspace connectivity health checks using the Azure.Analytics.Synapse.Artifacts client library
  • Multiple client creation modes for flexible authentication:
    • ServiceProvider (for pre-configured clients)
    • DefaultAzureCredentials (using Azure Identity)
    • ConnectionString (with workspace URI extraction)
  • Configurable timeout settings for health check operations
  • Comprehensive validation and error handling with proper exception management
  • Full unit test coverage including client creation scenarios and configuration validation

Implementation Details

The package provides a SynapseWorkspaceAvailableHealthCheck that tests connectivity to Azure Synapse Analytics workspaces by attempting to access the LinkedService client. This approach provides a lightweight connectivity check without requiring specific workspace resources.

The implementation follows the same architectural patterns as other Azure health check packages (Blobs, Tables, Queues) ensuring consistency across the codebase.

Usage

services.AddHealthChecks()
    .AddSynapseWorkspaceAvailability("synapse-check", options =>
    {
        options.WorkspaceUri = new Uri("https://myworkspace.dev.azuresynapse.net");
        options.Mode = SynapseClientCreationMode.DefaultAzureCredentials;
        options.Timeout = 5000;
    });

Fixes #42


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@samtrion
Copy link
Member

@copilot please finish this task

Copy link
Contributor Author

Copilot AI commented Aug 18, 2025

@copilot please finish this task

Task completed! I added the missing ClientCreationTests for the Azure Synapse health check package, completing the unit test coverage to match the patterns established by other Azure health check packages. The implementation now includes comprehensive tests for client creation scenarios. Commit 3b56d59.

@Copilot Copilot AI changed the title [WIP] NetEvolve.HealthChecks.Azure.Synapse NetEvolve.HealthChecks.Azure.Synapse - Complete implementation with unit tests Aug 18, 2025
Copilot finished work on behalf of samtrion August 18, 2025 21: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.

NetEvolve.HealthChecks.Azure.Synapse
2 participants