8000 Allow adaptation of the localstack run command in the supervisor by dfangl · Pull Request #11707 · localstack/localstack · GitHub 8000
[go: up one dir, main page]

Skip to content

Conversation

@dfangl
Copy link
Member
@dfangl dfangl commented Oct 17, 2024

Motivation

In order to start LocalStack from different main methods, we have to make the command the localstack supervisor starts configurable.

This will allow us to be more flexible in the way we start LocalStack.

Changes

  • Add LOCALSTACK_SUPERVISOR_COMMAND to change the command executed by the supervisor to start LocalStack (the main entrypoint).

@dfangl dfangl added the semver: patch Non-breaking changes which can be included in patch releases label Oct 17, 2024
@dfangl dfangl self-assigned this Oct 17, 2024
@dfangl dfangl requested a review from thrau October 17, 2024 12:53
@github-actions
Copy link

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 42m 32s ⏱️ -1s
3 508 tests ±0  3 095 ✅ ±0  413 💤 ±0  0 ❌ ±0 
3 510 runs  ±0  3 095 ✅ ±0  415 💤 ±0  0 ❌ ±0 

Results for commit d418f39. ± Comparison against base commit bf8fafe.

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! let's leave this undocumented for now.

works nicely:

 % LOCALSTACK_SUPERVISOR_COMMAND="echo 'foo'" bin/localstack-supervisor
foo
 % LOCALSTACK_SUPERVISOR_COMMAND="python -m localstack.cli.main start --host"  DNS_ADDRESS=0 bin/localstack-supervisor

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

 💻 LocalStack CLI 3.8.2.dev34
 👤 Profile: default

[19:00:18] starting LocalStack in host mode 💻                                                                                                                                localstack.py:501

LocalStack version: 3.8.2.dev34

Ready.

@thrau thrau merged commit 5c8320a into master Oct 17, 2024
@thrau thrau deleted the supervisor/configurable-command branch October 17, 2024 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver: patch Non-breaking changes which can be included in patch releases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

0