Skip to content

Add a note to Web App with SQL Database sample on MS EULA - #136

Merged
paolosalvatori merged 8 commits into
mainfrom
web-app-sql-db-note
Oct 14, 2026
Merged

paolosalvatori merged 8 commits into
mainfrom
web-app-sql-db-note

Conversation

@paolosalvatori

@paolosalvatori paolosalvatori commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Motivation

This PR fixes the issue #37 on the LocalStack for Azure Samples repository.

Fixes SMF-943

Changes

Add the following note to all the README files of the Web App with SQL Database (Python, .NET) samples:

Note

Each logical server you create runs a Microsoft SQL Server container, so you must accept the SQL Server End-User Licensing Agreement before the emulator provisions one.
Start the emulator with MSSQL_ACCEPT_EULA set to Y:

LOCALSTACK_MSSQL_ACCEPT_EULA=Y lstk start

Host variables prefixed with LOCALSTACK_ are forwarded to the emulator, so the container receives MSSQL_ACCEPT_EULA=Y.

To accept it once rather than on every start, add it to an environment profile in your lstk config:

[[containers]]
type = "azure"
tag  = "latest"
port = "4566"
env  = ["mssql"]

[env.mssql]
MSSQL_ACCEPT_EULA = "Y"

A profile's values are injected into the container directly, so the LOCALSTACK_ prefix is not used here.
lstk reads the first config.toml it finds, starting with ./.lstk/config.toml in the current directory and then $HOME/.config/lstk/config.toml.
See Config file search order for the full list and Passing environment variables to the container for how profiles work.

If you start the emulator with the docker CLI or Docker Compose, set MSSQL_ACCEPT_EULA=Y directly in the container environment instead.
Without it, az sql server create fails with (EulaNotAccepted).

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.

🟡 Changes recommended

The startup commands do not explicitly select Azure, and the new documentation links resolve incorrectly from GitHub.

8 open findings
What changed in this PR

Adds SQL Server EULA setup guidance throughout the Python and .NET Web App with SQL Database samples.

Changes:

  • Documents one-time and persistent EULA configuration.
  • Covers lstk, Docker CLI, and Docker Compose workflows.
File Description
python/​README.md Adds Python sample guidance.
python/​bicep/​README.md Adds Bicep guidance.
python/​scripts/​README.md Adds script guidance.
python/​terraform/​README.md Adds Terraform guidance.
dotnet/​README.md Adds .NET sample guidance.
dotnet/​bicep/​README.md Adds Bicep guidance.
dotnet/​scripts/​README.md Adds script guidance.
dotnet/​terraform/​README.md Adds Terraform guidance.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread samples/web-app-sql-database/dotnet/README.md Outdated
> Start the emulator with `MSSQL_ACCEPT_EULA` set to `Y`:
>
> ```bash
> LOCALSTACK_MSSQL_ACCEPT_EULA=Y lstk start
Comment thread samples/web-app-sql-database/dotnet/scripts/README.md Outdated
Comment thread samples/web-app-sql-database/dotnet/terraform/README.md Outdated
Comment thread samples/web-app-sql-database/python/README.md Outdated
Comment thread samples/web-app-sql-database/python/bicep/README.md Outdated
Comment thread samples/web-app-sql-database/python/scripts/README.md Outdated
Comment thread samples/web-app-sql-database/python/terraform/README.md Outdated
paolosalvatori and others added 7 commits October 14, 2026 15:23
Clarify instructions for starting the emulator with EULA acceptance.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Clarify the command to start the emulator with EULA acceptance.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Updated instructions for starting the emulator with EULA acceptance.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Clarified instructions for starting the emulator with EULA acceptance.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@paolosalvatori
paolosalvatori merged commit 6fdecc1 into main Oct 14, 2026
8 checks passed

This branch was successfully deployed

1 active deployment
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.

2 participants