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.

Sign up for GitHub

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

Jump to bottom

Update Azure Sql to use new API pattern #5998

Closed
eerhardt opened this issue Sep 27, 2024 · 0 comments · Fixed by #6094
Closed

Update Azure Sql to use new API pattern #5998

eerhardt opened this issue Sep 27, 2024 · 0 comments · Fixed by #6094
Assignees
@eerhardt
Labels
area-integrations Issues pertaining to Aspire Integrations packages azure Issues associated specifically with scenarios tied to using Azure
Milestone
9.0

Comments

@eerhardt
Copy link
Member

With

We are moving to a new API pattern for Azure resources that can also be used locally as a container. We should update Azure Sql Server to follow the same pattern.

  1. The existing [Publish]AsAzureSqlDatabase methods will be marked Obsolete
  2. New AddAzureSqlServer method will be introduced for when you want an Azure resource.
  3. To use a non-Azure container locally, a new RunAsContainer method can be called.
@eerhardt eerhardt added this to the 9.0 milestone Sep 27, 2024
@eerhardt eerhardt added area-integrations Issues pertaining to Aspire Integrations packages azure Issues associated specifically with scenarios tied to using Azure enhancement labels Sep 27, 2024
This was referenced Sep 27, 2024
Closed
Merged
Open
@eerhardt eerhardt self-assigned this Oct 1, 2024
eerhardt added a commit to eerhardt/aspire that referenced this issue Oct 4, 2024
@eerhardt
Update Azure Sql to use new API pattern
8de6447 
We are moving to a new API pattern for Azure resources that can also be used locally as a container.Update Azure Sql Server to follow the same pattern.

* The existing [Publish]AsAzureSqlDatabase methods are marked Obsolete
* New AddAzureSqlServer method is introduced for when you want an Azure resource.
* To use a non-Azure container locally, a new RunAsContainer method is added.

Fix dotnet#5998
@eerhardt eerhardt mentioned this issue Oct 4, 2024
Merged
7 tasks
eerhardt added a commit that referenced this issue Oct 4, 2024
@eerhardt
Update Azure Sql to use new API pattern (#6094)
07f1f34 
* Update Azure Sql to use new API pattern

We are moving to a new API pattern for Azure resources that can also be used locally as a container.Update Azure Sql Server to follow the same pattern.

* The existing [Publish]AsAzureSqlDatabase methods are marked Obsolete
* New AddAzureSqlServer method is introduced for when you want an Azure resource.
* To use a non-Azure container locally, a new RunAsContainer method is added.

Fix #5998

* Playground manifest updates

* Fix stack overflow.
@github-actions github-actions bot locked and limited conversation to collaborators Nov 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@eerhardt eerhardt

Labels
area-integrations Issues pertaining to Aspire Integrations packages azure Issues associated specifically with scenarios tied to using Azure
Projects
None yet
Milestone
9.0
Development

Successfully merging a pull request may close this issue.

Update Azure Sql to use new API pattern eerhardt/aspire
1 participant
@eerhardt