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

Rewrite SQL Server integration #2086

Merged
merged 16 commits into from
Nov 20, 2024
Merged

Conversation

IEvangelist
Copy link
Member

@IEvangelist IEvangelist commented Nov 13, 2024

@IEvangelist IEvangelist requested a review from gewarren November 13, 2024 20:16
@dotnetrepoman dotnetrepoman bot added the database Content related to database. label Nov 13, 2024
@dotnet-policy-service dotnet-policy-service bot added the okr-quality Content-health KR: Concerns article defects/freshness or build warnings. label Nov 13, 2024
@davidfowl
Copy link
Member

davidfowl commented Nov 14, 2024

Some details that I think we should add (from #2020):

  • Show how to use SSMS and Azure Data Studio with aspire SQL server containers
    • This includes how to fix the port and potentially the password in the app host. If we don't fix the password then the doc needs to show how to find the password for the SQL container in the dashboard UI
  • Tell users that the password is stored in the volume. If you use a volume and change the password, it will not work until you delete the volume. (This should be a big warning on each of the database providers, pg and sql).
  • Mention in a note that the SQL server container is slow to start and is best used together with persistent containers.

@gewarren
Copy link
Contributor

I'll wait to review till you add the requested details.

@IEvangelist
Copy link
Member Author

@davidfowl - I'm attempting to generalize the data-access content, so I'd like to know, would the work here for the SSMS and Azure Data Studio also be applicable to the other DB providers we integrate with? If so, that would be really good to know now.

@davidfowl
Copy link
Member

SSMS, no, that's SQL Server specific. Azure Data Studio I think works for others but I haven't spent much time it.

@IEvangelist IEvangelist merged commit a71777b into dotnet:main Nov 20, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
database Content related to database. okr-quality Content-health KR: Concerns article defects/freshness or build warnings.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rewrite of the SQL Server integration articles
3 participants