Skip to content

Conversation

@captainsafia
Copy link
Member

Azure Tables are a dependency for Durable Functions in deployments. We need to implement the IResourceWithAzureFunctionsConfig on the AzureStorageTableResource in order to support this.

Copilot AI review requested due to automatic review settings March 8, 2025 01:00
Copy link
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.

PR Overview

This PR adds support for Azure Functions configuration for the Azure Table Storage resource by implementing the IResourceWithAzureFunctionsConfig interface. Key changes include:

  • Updating AzureTableStorageResource to implement IResourceWithAzureFunctionsConfig.
  • Adding an explicit implementation of ApplyAzureFunctionsConfiguration in AzureTableStorageResource.
  • Introducing Tables-specific connection key prefixes and settings in AzureStorageResource.

Reviewed Changes

File Description
src/Aspire.Hosting.Azure.Storage/AzureTableStorageResource.cs Added IResourceWithAzureFunctionsConfig implementation to provide Azure Functions configuration support for table storage.
src/Aspire.Hosting.Azure.Storage/AzureStorageResource.cs Introduced TablesConnectionKeyPrefix constant and updated ApplyAzureFunctionsConfiguration to include table-related settings.

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

@davidfowl
Copy link
Member

No test updates?

@captainsafia
Copy link
Member Author

No test updates?

Our test coverage for these has usually been E2E tests with a trigger. We don't have unit tests for this behavior.

I can add some in a follow-up PR.

@captainsafia captainsafia merged commit c4e4092 into main Mar 8, 2025
137 checks passed
@captainsafia captainsafia deleted the cs/funcs-table-storage branch March 8, 2025 05:17
@github-actions github-actions bot added the area-integrations Issues pertaining to Aspire Integrations packages label Mar 10, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Apr 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-integrations Issues pertaining to Aspire Integrations packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants