Skip to content

Conversation

@roberto-mardeni
Copy link
Contributor

Replaced MSSQL_SA_PASSWORD for SA_PASSWORD

Replaced MSSQL_SA_PASSWORD for SA_PASSWORD
@PRMerger20
Copy link
Contributor

@roberto-mardeni : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@VanMSFT
Copy link
Member

VanMSFT commented Apr 7, 2020

Hello @roberto-mardeni! Thank you for this submission! Can you please provide some details for this change request?

@ktoliver ktoliver added the aq-pr-triaged tracking label for the PR review team label Apr 8, 2020
@roberto-mardeni
Copy link
Contributor Author

Because the newest container image available at https://hub.docker.com/_/microsoft-mssql-server indicates that the environment variable for the password is SA_PASSWORD and not MSSQL_SA_PASSWORD

@VanMSFT
Copy link
Member

VanMSFT commented Apr 8, 2020

Thanks, @roberto-mardeni!

@vin-yu - Can you please help confirm the change?

@VanMSFT
Copy link
Member

VanMSFT commented Apr 10, 2020

Confirmed with Vin.

Thanks @roberto-mardeni! We appreciate your assistance on our documentation!

#sign-off

@ktoliver ktoliver merged commit 2fdd6a5 into MicrosoftDocs:live Apr 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants