Rename SA_PASSWORD to MSSQL_SA_PASSWORD as SA_PASSWORD is deprecated

[jedla97]

Updating password env variable for MSSQL as it was deprecated see https://learn.microsoft.com/en-us/sql/linux/quickstart-install-connect-docker?view=sql-server-ver16&tabs=cli&pivots=cs1-bash#run-the-container-2

[quarkus-bot]

/cc @gsmet (hibernate-orm), @yrodiere (hibernate-orm)

[yrodiere]

Thanks, LGTM.

From what I can see it should work on both SQL Server 2022 and azure-sql-edge:latest (used for Mac M1 testing)

I checked dev services, and there the environment variable is set by testcontainers itself, so we can let them do the update (no idea if they already did).

[quarkus-bot]

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 9cd76a0.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

[jedla97]

Thanks for looking into it. What I was looking at docs for MSSQL 2017 the SA_PASSWORD note is also there and I was able to start docker image with the MSSQL_SA_PASSWORD so this probably happend long time ago but both still work.

Looking at the testcointainers repo they still using old https://github.com/testcontainers/testcontainers-java/blob/main/modules/mssqlserver/src/main/java/org/testcontainers/containers/MSSQLServerContainer.java#L81