-
Notifications
You must be signed in to change notification settings - Fork 214
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
Add support for missing resource types #3020
Comments
+1 on getting the SqlServer resource type supported. Currently hitting:
ASIDE: Is it a bug that the sqlserver.server resource type is v1 (set in WriteSqlServerContainerToManifest() in SqlServerBuilderExtensions.cs). Should it be v0 (like all the others) |
Is implementing the container resources for items not supported by "az appcontainer add-on create" going to be hard? For example, will azd wait until "az appcontainer add-on sqlserver" support is added before trying to implement sqlserver.server.v1 support. My understanding is, without appcontainer add-on support for sqlserver, azd will need to do the work to config Ingress, setup a container secret (the mssql equiv to the pg-password), and also pass the required environment variables (e.g. ACCEPT_EULA and MSSQL_SA_PASSWORD). While sqlserver is not supported by "appcontainer add-ons", will azd still move forward in supporting sqlserver.server.v1 (and do all the wiring up (Ingress/Secret/EnvVars) required in azd), or will azd wait until "az appcontianer add-on" support is added for "sqlserver"? |
@stuartpa , azd interacts with appcontainer service directly, using the sdk clients. So, no, what |
Okay, same question rephrased: While sqlserver is not supported by "appcontainer add-ons", will azd still move forward in supporting sqlserver.server.v1 (and do all the wiring up (Ingress/Secret/EnvVars) required in azd), or will azd wait until "az appcontianer add-on" support is added for "sqlserver"? |
We are working through these details at the moment. Our goal is to have it so that when azd sees a sqlserver resource that it will deploy an Azure SQL Database instance. |
@eerhardt we need to add kafka to this list. ACA's kafka support https://learn.microsoft.com/en-us/azure/container-apps/tutorial-dev-services-kafka?tabs=bash |
Note I can't edit the top list, as I'm not a contributor in this repo. I noticed a couple more missing as well. I've opened the following issues, if you want to add them to this list @ellismg: |
@mitchdenny --> Instead of creating an 'Azure SQL Database instance,' could you set up an entire AKS with two PODs—one for SQL Server and another for the web? I have already accomplished this, and I can provide the YAML files if needed. |
There's no support for AKS in aspire via azd for v1. We're primarily targeting ACA. aspirate can deploy SQL containers to Kubernetes. |
Also running into the Would love to automatically have everything provisioned 😁 |
SQL Server will work in the next preview of azd and aspire. |
We will not add hand-crafted, template-based Bicep deployment support in azd. Instead, we will be moving towards using bicep resource type and CDK effort. |
https://learn.microsoft.com/en-us/dotnet/aspire/deployment/manifest-format#built-in-resources
https://learn.microsoft.com/en-us/dotnet/aspire/deployment/manifest-format#azure-specific-resource-types
The text was updated successfully, but these errors were encountered: