Skip to content

.Net: Use testcontainer for SQL Server 2025 #13465

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
Merged
markwallace-microsoft merged 3 commits into from
Jan 26, 2026
Merged

.Net: Use testcontainer for SQL Server 2025 #13465

markwallace-microsoft merged 3 commits into from
Jan 26, 2026
+240 −62

Conversation

@roji
Copy link
Member

@roji roji commented Jan 22, 2026

And allow pointing to an external database via environment variable on SQL Server and PostgreSQL for faster iteration.

Test-only change.

@roji
Use testcontainer for SQL Server 2025
f2503ab 
And allow pointing to an external database via environment
variable on SQL Server and PostgreSQL for faster iteration.
@roji roji requested a review from a team as a code owner January 22, 2026 11:58
@moonbox3 moonbox3 added .NET Issue or Pull requests regarding .NET code documentation labels Jan 22, 2026
@github-actions github-actions bot changed the title Use testcontainer for SQL Server 2025 .Net: Use testcontainer for SQL Server 2025 Jan 22, 2026
@roji roji enabled auto-merge January 22, 2026 21:52
@roji
Copy link
Member Author

roji commented Jan 22, 2026

Thanks for the review @westey-m, have added user secrets support.

@roji roji added this pull request to the merge queue Jan 26, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 26, 2026
@roji roji added this pull request to the merge queue Jan 26, 2026
github-merge-queue bot pushed a commit that referenced this pull request Jan 26, 2026
@roji @markwallace-microsoft
.Net: Use testcontainer for SQL Server 2025 (#13465)
eb1f6ce 
And allow pointing to an external database via environment variable on
SQL Server and PostgreSQL for faster iteration.

Test-only change.

---------

Co-authored-by: Mark Wallace <127216156+markwallace-microsoft@users.noreply.github.com>
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 26, 2026
@roji roji added this pull request to the merge queue Jan 26, 2026
@markwallace-microsoft
Copy link
Member

markwallace-microsoft commented Jan 26, 2026

Unrelated integration test failure

  Passed SemanticKernel.IntegrationTests.Connectors.AzureOpenAI.AzureOpenAITextToImageTests.GetImageContentsCanReturnImageUrlAsync [9 s]
[xUnit.net 00:05:21.74]     SemanticKernel.IntegrationTests.Connectors.AzureOpenAI.AzureOpenAITextToImageTests.ItCanReturnImageUrlAsync [FAIL]
[xUnit.net 00:05:21.75]       Microsoft.SemanticKernel.HttpOperationException : HTTP 400 (invalid_request_error: content_policy_violation)
[xUnit.net 00:05:21.75]       
[xUnit.net 00:05:21.75]       Your request was rejected as a result of our safety system. Your prompt may contain text that is not allow

@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 26, 2026
@markwallace-microsoft markwallace-microsoft merged commit 3ce1a84 into microsoft:main Jan 26, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@markwallace-microsoft markwallace-microsoft markwallace-microsoft approved these changes

@westey-m westey-m westey-m approved these changes

Assignees

No one assigned

Labels

documentation .NET Issue or Pull requests regarding .NET code

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@roji @markwallace-microsoft @westey-m @moonbox3