Fixed MSSQL startup #75
Conversation
| @@ -5,7 +5,7 @@ | |||
|
|
|||
| for i in {1..50}; | |||
| do | |||
| /opt/mssql-tools/bin/sqlcmd -S localhost -U sa -P $SA_PASSWORD -d master -Q "CREATE DATABASE $MSSQL_DATABASE;" | |||
| /opt/mssql-tools18/bin/sqlcmd -S localhost -U sa -P $SA_PASSWORD -No -d master -Q "CREATE DATABASE $MSSQL_DATABASE;" | |||
There was a problem hiding this comment.
@iliayatsenko Do you have some reference about this renaming?
Could this be the reason for the failing pipeline in #77 (comment) as well?
There was a problem hiding this comment.
@W0rma , yes, all description is in related issue. In short, please see this comment - microsoft/mssql-docker#892 (comment)
Regarding #77 , yes, it looks like the reason is the same.
There was a problem hiding this comment.
I suggest replacing the MSSQL image with server:2022-CU13-ubuntu-20.04, as the path issue is fixed in this version. This change will help avoid breaking the CI again if they revert the path in the future.
You can replace the image in docker_compose.yaml and github action configuration.
I've tested it on my branch and local machine, and it works properly.
Closes #74