Skip to content

deprecate starting in host mode via the CLI #13398

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
alexrashed merged 2 commits into from
Nov 19, 2025
Merged

deprecate starting in host mode via the CLI #13398

alexrashed merged 2 commits into from
Nov 19, 2025

Conversation

@alexrashed
Copy link
Member

@alexrashed alexrashed commented Nov 19, 2025 •
edited
Loading

Motivation

We decided to finally separate the CLI from the runtime packages, because:

  • We want to decouple the releases of the CLI from the releases of the runtime. This will allow us to iterate faster and react on issues faster.
  • The CLI has completely different requirements (f.e. with which Python versions it needs to be compatible) causing all kinds of issues and complex setup (f.e. with the linting, the imports, etc).
  • The tight coupling between the code of the CLI and the runtime is causing issues due to unwanted imports, complex coupling between the CLI and the configuration values / runtime versions it supports, etc.

In order to prepare the extraction of the CLI and the decoupling from the runtime, we are deprecating the host mode of the CLI.
In future releases, it will not be possible to run localstack start --host anymore and we directly refer users to the default way of executing LocalStack via the CLI (which starts a Docker container).

With this PR a prominent deprecation warning is being introduced to warn about these upcoming changes:

❯ localstack start --host
DeprecationWarning: The option 'host' is deprecated.

     __                     _______ __             __
    / /   ____  _________ _/ / ___// /_____ ______/ /__
   / /   / __ \/ ___/ __ `/ /\__ \/ __/ __ `/ ___/ //_/
  / /___/ /_/ / /__/ /_/ / /___/ / /_/ /_/ / /__/ ,<
 /_____/\____/\___/\__,_/_//____/\__/\__,_/\___/_/|_|

- LocalStack CLI: 4.10.1.dev37
- Profile: default
- App: https://app.localstack.cloud

[09:51:35] starting LocalStack in host mode 💻
           Warning: Starting LocalStack in host mode from the LocalStack CLI has been deprecated!
           This feature will be removed soon! Please use the Docker mode (default) instead!

Where the DeprecationWarning on top by Click is red, and the manual deprecation warning below is red and bold.

Changes

  • Explicitly deprecate the --host option of the start command of the LocalStack CLI.
  • If the host mode is used, show a very prominent deprecation warning.

Fixes ENG-138.

@alexrashed alexrashed added this to the 4.11 milestone Nov 19, 2025
@alexrashed alexrashed requested a review from silv-io November 19, 2025 08:59
@alexrashed alexrashed requested a review from thrau as a code owner November 19, 2025 08:59
@alexrashed alexrashed added area: cli LocalStack CLI semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases docs: needed Pull request requires documentation updates notes: needed Pull request should be mentioned in the release notes review: merge when ready Signals to the reviewer that a PR can be merged if accepted labels Nov 19, 2025
@alexrashed alexrashed mentioned this pull request Nov 19, 2025
Merged
@github-actions
Copy link

github-actions bot commented Nov 19, 2025 •
edited
Loading

S3 Image Test Results (AMD64 / ARM64)

    2 files    2 suites   8m 10s ⏱️
  535 tests 483 ✅  52 💤 0 ❌
1 070 runs  966 ✅ 104 💤 0 ❌

Results for commit 5faf6e4.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Nov 19, 2025 •
edited
Loading

Test Results - Preflight, Unit

22 332 tests  ±0   20 578 ✅ ±0   6m 32s ⏱️ +7s
     1 suites ±0    1 754 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 5faf6e4. ± Comparison against base commit 7aa5ff9.

♻️ This comment has been updated with latest results.

thrau
thrau approved these changes Nov 19, 2025
View reviewed changes
Copy link
Member

@thrau thrau left a comment

Choose a reason for hiding this comment

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

LGTM! I didn't follow the decision, but trust that we went through the appropriate product questions (are we impacting users, are there any important accounts using the feature, and if so how can we migrate them, etc.)

@github-actions
Copy link

github-actions bot commented Nov 19, 2025 •
edited
Loading

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 19s ⏱️ -1s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 5faf6e4. ± Comparison against base commit 7aa5ff9.

♻️ This comment has been updated with latest results.

@alexrashed @thrau
update deprecation warning
5faf6e4 
Co-authored-by: Thomas Rausch <[email protected]>
@alexrashed
Copy link
Member Author

alexrashed commented Nov 19, 2025

Thanks, @thrau! Before pushing for this change, I did check the GitHub search to find usages of the host mode in the CLI. There are only very few actual usages, most of the search results are old forks of our own code or very outdated repos. The remaining usages should be easy to migrate to the Docker mode. 👍🏽

silv-io
silv-io approved these changes Nov 19, 2025
View reviewed changes
Copy link
Member

@silv-io silv-io left a comment

Choose a reason for hiding this comment

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

LGTM! Do we have some telemetry in place to track usage of it going forward? This way we can better judge when to fully remove that option.

@github-actions
Copy link

github-actions bot commented Nov 19, 2025

Test Results (amd64) - Integration, Bootstrap

    5 files  ±0      5 suites  ±0   2h 40m 39s ⏱️ +33s
5 315 tests ±0  4 776 ✅ ±0  539 💤 ±0  0 ❌ ±0 
5 321 runs  ±0  4 776 ✅ ±0  545 💤 ±0  0 ❌ ±0 

Results for commit 5faf6e4. ± Comparison against base commit 7aa5ff9.

@github-actions
Copy link

github-actions bot commented Nov 19, 2025

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   2h 1m 5s ⏱️ - 3m 59s
4 941 tests ±0  4 562 ✅ ±0  379 💤 ±0  0 ❌ ±0 
4 943 runs  ±0  4 562 ✅ ±0  381 💤 ±0  0 ❌ ±0 

Results for commit 5faf6e4. ± Comparison against base commit 7aa5ff9.

@alexrashed alexrashed merged commit dba528e into main Nov 19, 2025
49 checks passed
@alexrashed alexrashed deleted the deprecate-cli-host-mode branch November 19, 2025 12:59
@alexrashed alexrashed mentioned this pull request Nov 26, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@thrau thrau thrau approved these changes

@silv-io silv-io silv-io approved these changes

Assignees

@thrau thrau

Labels

area: cli LocalStack CLI docs: needed Pull request requires documentation updates notes: needed Pull request should be mentioned in the release notes review: merge when ready Signals to the reviewer that a PR can be merged if accepted semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases

Projects

None yet

Milestone

4.11

Development

Successfully merging this pull request may close these issues.

4 participants

@alexrashed @thrau @silv-io