Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Certify Azure Table Storage as Stable in state store #950

Closed
artursouza opened this issue Jun 17, 2021 · 8 comments · Fixed by #1816
Closed

Certify Azure Table Storage as Stable in state store #950

artursouza opened this issue Jun 17, 2021 · 8 comments · Fixed by #1816
Assignees
Milestone

Comments

@artursouza
Copy link
Member

Describe the feature

Certify Azure Table Storage as GA in state store

Release Note

RELEASE NOTE:

@artursouza artursouza added this to Backlog in Dapr Roadmap via automation Jun 17, 2021
@greenie-msft greenie-msft removed this from Backlog in Dapr Roadmap Jun 23, 2021
@artursouza artursouza added this to the v1.4 milestone Jul 28, 2021
@artursouza artursouza added this to Backlog in Dapr Roadmap via automation Jul 28, 2021
@artursouza artursouza moved this from Backlog to Planned (Committed) in Dapr Roadmap Jul 28, 2021
@greenie-msft greenie-msft moved this from Planned (Committed) to In Progress (Development) in Dapr Roadmap Aug 2, 2021
@artursouza artursouza modified the milestones: v1.4, v1.5 Aug 26, 2021
@berndverst berndverst removed their assignment Oct 12, 2021
@tanvigour
Copy link
Contributor

/assign

@tanvigour
Copy link
Contributor

tanvigour commented Oct 13, 2021

Test Plan Draft

Proposed tests (Integration)

@artursouza artursouza modified the milestones: v1.5, v1.6 Oct 26, 2021
@berndverst
Copy link
Member

berndverst commented Nov 16, 2021

@tanvigour @artursouza important update:

This component should switch to the new / current Track 2 SDK for Table Storage.

This SDK supports BOTH the Azure Table Storage (as part of a Storage Container) as well as the Cosmos DB Table API.

https://github.com/Azure/azure-sdk-for-go/tree/main/sdk/data/aztables

IMO we should not certify as stable this component using the existing outdated SDK

@berndverst berndverst modified the milestones: v1.6, v1.7 Dec 9, 2021
@artursouza artursouza removed this from the v1.7 milestone Feb 1, 2022
@artursouza artursouza moved this from In Progress (Development) to Backlog in Dapr Roadmap Feb 1, 2022
@berndverst
Copy link
Member

From recent Discord activity is clear that customers expect this component to connect to both Azure Blobstorage Tables as well as Cosmos DB Tables API. That can be facilitated by using the new Track 2 SDK and exposing the relevant options as component metadata.

Tracking the Track 2 migration work separately at #1528

@artursouza artursouza added this to the v1.8 milestone Mar 3, 2022
@tanvigour
Copy link
Contributor

tanvigour commented Mar 21, 2022

From recent Discord activity is clear that customers expect this component to connect to both Azure Blobstorage Tables as well as Cosmos DB Tables API. That can be facilitated by using the new Track 2 SDK and exposing the relevant options as component metadata.

Tracking the Track 2 migration work separately at #1528

Do we have an estimate on completion of migration of table storage state store to track 2 sdk?

@tanvigour
Copy link
Contributor

tanvigour commented Apr 1, 2022

Test Plan Draft

Basic Test for created table for CRUD operations:

  1. Able to create and test connection.
  2. Able to do set, fetch, delete.

Basic Test for non-created table for CRUD operations:

  1. Able to create and test connection.
  2. Able to do set, fetch, delete data.
  3. Delete the table

Test for authentication using Azure Auth layer

  1. Save Data using Go SDK with already existing table
  2. Retrieve data by using AAD credentials

@tanvigour
Copy link
Contributor

@artursouza Please review the test plan when you get a chance. Thanks!

@mukundansundar
Copy link
Contributor

@tanvigour

  • Add tests for concurrency
  • Add tests for etag support
  • Test all CRUD operations ... as in add in tests for update and delete as well
  • Add tests for the negative scenarios as well, like deleting or reading non-existent keys etc.

@mukundansundar mukundansundar changed the title Certify Azure Table Storage as GA in state store Certify Azure Table Storage as Stable in state store Apr 2, 2022
@artursouza artursouza moved this from Backlog to Planned (Committed) in Dapr Roadmap Apr 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Dapr Roadmap
Planned (Committed)
Development

Successfully merging a pull request may close this issue.

4 participants