Skip to content

Comments

fix: correct LocalStack status after Lambda invocation#7

Merged
skyrpex merged 1 commit intomainfrom
fix-localstack-status-tracker
Sep 2, 2025
Merged

fix: correct LocalStack status after Lambda invocation#7
skyrpex merged 1 commit intomainfrom
fix-localstack-status-tracker

Conversation

@skyrpex
Copy link
Collaborator

@skyrpex skyrpex commented Sep 2, 2025

LocalStack’s status tracker was incorrectly reporting the main container as stopped after invoking a Lambda function. This happened because ‎docker events --filter container=localstack-main matches any container whose name starts with ‎localstack-main, including temporary Lambda containers (e.g., ‎localstack-main-<hash>). As a result, the tracker picked up stop events for these Lambda containers instead of the main one, leading to false status reports. This commit updates the filter logic to ensure only the main container’s events are tracked.

LocalStack’s status tracker was incorrectly reporting the main container as stopped after invoking a Lambda function. This happened because ‎`docker events --filter container=localstack-main` matches any container whose name starts with ‎`localstack-main`, including temporary Lambda containers (e.g., ‎`localstack-main-<hash>`). As a result, the tracker picked up stop events for these Lambda containers instead of the main one, leading to false status reports. This commit updates the filter logic to ensure only the main container’s events are tracked.
Copy link
Collaborator

@tiurin tiurin left a comment

Choose a reason for hiding this comment

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

👏 great finding @skyrpex!

@skyrpex skyrpex merged commit 304f1f4 into main Sep 2, 2025
2 checks passed
@tiurin tiurin mentioned this pull request Sep 2, 2025
tiurin added a commit that referenced this pull request Sep 2, 2025
@tiurin tiurin deleted the fix-localstack-status-tracker branch September 3, 2025 09:20
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