-
Notifications
You must be signed in to change notification settings - Fork 308
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
Allow connection names in database components #30
Conversation
Make database components consistent with Redis connection string structure. - Add a string connectionName parameter to AddRedis - Rename the keyed DI methods to .AddKeyedRedis. - keep just the string name for now. - Hosting changes to pass the component's name as the ConnectionStrings__{name} env var
src/Aspire.Hosting/Postgres/PostgresContainerBuilderExtensions.cs
Outdated
Show resolved
Hide resolved
src/Aspire.Hosting/Postgres/PostgresContainerBuilderExtensions.cs
Outdated
Show resolved
Hide resolved
src/Aspire.Hosting/SqlServer/SqlServerCloudApplicationBuilderExtensions.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few minor nits but otherwise good to see this get finalized!
Do we update the configuration schema for this too? |
There aren't any changes to the configuration schema in this change. |
I've added tests and updated the READMEs. This should be ready for review now. |
* Icons for Aspire.Hosting.Azure packages Using Azure Icons from Brand Central #3077 * Added readme files for Azure hosting packages #3077 Related Work Items: #30 * Fixup of Azure components readme - fixed references to now use split up package names * Fixing missed typo correction for CosmosDbClient * PR feedback: reference client libraries
(In draft until I update READMEs and add tests for the new parameter).
Make database components consistent with Redis connection string structure.
Note: I also removed the SqlServer database from the sample app, since it doesn't do anything and only slows down and complicates the app.