Skip to content

V15 QA added a fix for flaky integration tests run on SQL Server Linux - #18965

Merged
andr317c merged 17 commits into
v15/devfrom
v15/QA/fix-integration-tests-handshake-pipeline-issue
May 8, 2025
Merged

V15 QA added a fix for flaky integration tests run on SQL Server Linux#18965
andr317c merged 17 commits into
v15/devfrom
v15/QA/fix-integration-tests-handshake-pipeline-issue

Conversation

@andr317c

@andr317c andr317c commented Apr 8, 2025

Copy link
Copy Markdown
Contributor

This PR fixes the issue we have seen on the pipeline for SQL Server Integration tests running on Linux. It adds a check to ensure the SQL Server Docker Image is ready.

The issue was :
Error Message: Microsoft.Data.SqlClient.SqlException : A connection was successfully established with the server, but then an error occurred during the pre-login handshake. (provider: TCP Provider, error: 0 - Success)

It only happens sometimes, which is why I suspect the reason for this happening is that we do not wait till the SQL Server Docker Image is ready.

Copilot AI review requested due to automatic review settings April 8, 2025 07:14

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread build/azure-pipelines.yml Outdated

@nhudinh0309 nhudinh0309 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

…tion-tests-handshake-pipeline-issue

# Conflicts:
#	build/nightly-E2E-test-pipelines.yml
@andr317c
andr317c merged commit b01def0 into v15/dev May 8, 2025
@andr317c
andr317c deleted the v15/QA/fix-integration-tests-handshake-pipeline-issue branch May 8, 2025 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants